Re: [AD] fshook changes

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


On January 11, 2009, Thomas Fjellstrom wrote:
> On January 10, 2009, Thomas Fjellstrom wrote:
> > Over the past few months I've gotten some feedback which is good, not as
> > much as I was hoping for, which is bad :(
> >
> > But anyhow I have a growing TODO list for the file system code that I'd
> > like to share, and see if theres anything I'm forgetting.
> >
> > In no particular order:
> > - fix up rest of documentation, and resolve XXX comments
> > - change some functions to return bool, and fill in errno as necessary
> > - implement missing functions: al_find*, abs/rel/cannon conversions
> > - rename al_fs_[im](get|put)wl to al_readbitnum[lb]
> >   - al_fs_igetw -> al_read16l
> > - AL_ -> ALLEGRO_
> > - al_fs_readdir: have it fill in a ALLEGRO_ENTRY struct
> > - convert remaining parts of allegro to fshooks
> >   - config.c, icodec, acodec (some is done already)
> > - bulk rename of api to a more a5 consistent scheme:
> >   - al_fs_create_handle -> al_create_entry
> >   - al_fs_entry_name -> al_get_entry_name
> >   - al_fs_exists -> al_exists
> > - add a al_set_appname function (to be used in al_get_path, and possibly
> > other places)
> >
> > The bulk rename is currently at the bottom of my list, I figure it can
> > wait till the rest is done.
> >
> > So if anyone has any comments or suggestions, PLEASE pass them along.
>
> Something new to think about, I'm sure we want large file support on 32bit
> systems, so at least on linux (and some unixs) we need to use fseeko and
> ftello instead of the non o versions.
>
> they happen to be SUSv2, and POSIX.1-2001, is that enough? I was planning
> on adding a check in cmake for them, but the allegro api will change
> depending if they exist or not... fseek/ftell deal with 'long',
> fseeko/ftello deal with off_t (which is 64bit with the right defines set).
>
> So supporting both versions means incompatible dlls...

More precisely its an ABI break based on a build time check.

> > --
> > Thomas Fjellstrom
> > tfjellstrom@xxxxxxxxxx
> >
> > -------------------------------------------------------------------------
> >-- --- Check out the new SourceForge.net Marketplace.
> > It is the best place to buy or sell services for
> > just about anything Open Source.
> > http://p.sf.net/sfu/Xq1LFB
>
> --
> Thomas Fjellstrom
> tfjellstrom@xxxxxxxxxx
>
> ---------------------------------------------------------------------------
>--- Check out the new SourceForge.net Marketplace.
> It is the best place to buy or sell services for
> just about anything Open Source.
> http://p.sf.net/sfu/Xq1LFB


-- 
Thomas Fjellstrom
tfjellstrom@xxxxxxxxxx




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