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