Re: [AD] file interface fopen

[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]


On December 20, 2010, Matthew Leverton wrote:
> On Mon, Dec 20, 2010 at 2:07 AM, Peter Wang <novalazy@xxxxxxxxxx> wrote:
> > al_fopen_vt perhaps has a purpose.  I can't see one for
> > `al_create_file_handle(void *userdata)'.  A concern is that you might
> > accidentally pair the default interface with an unrelated userdata.
> 
> I don't think it's particularly useful either.
> 
> Attached is a new patch (against 5.1 svn). I finished it before I saw
> the above comment, so it includes al_create_file_handle_vt(). It can
> easily be renamed and moved with al_create_file_handle().
> 
> Otherwise, I think it's as dicussed.
> 
> --
> Matthew Leverton

I really do think it'd be better to make everything consistent. And not leave 
a bunch of compatibility code in long term for the old interface.

-- 
Thomas Fjellstrom
tfjellstrom@xxxxxxxxxx




Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/