Re: [AD] SF.net SVN: alleg:[13892] allegro/branches/4.9/src/macosx/system.m

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


On November 13, 2010, Elias Pschernig wrote:
> On Sat, 2010-11-13 at 19:07 +0100, Michał Cichoń wrote:
> > Is the ALLEGRO_PATH a path to the file or directory? I think both.
> > 
> > More proper names for _PATH variables are:
> > ALLEGRO_TEMP_DIR_PATH
> > ALLEGRO_DATA_DIR_PATH
> > ALLEGRO_SYSTEM_SETTINGS_DIR_PATH
> > ALLEGRO_USER_DATA_DIR_PATH
> > ALLEGRO_USER_SETTINGS_DIR_PATH
> > ALLEGRO_USER_HOME_DIR_PATH
> > ALLEGRO_EXENAME_PATH
> > 
> > Suffixing with _DIR_PATH should solve all doubts.
> 
> They sound a bit strange, so not sure it's really needed for the one
> exception we have.

Indeed. I see no reason to insert _DIR.
 
> > Also ALLEGRO_APPLICATION_PATH is suits better than ALLEGRO_EXENAME_PATH
> > IMHO.
> 
> I can only agree to that though.

But that is sensible. Unless we wanted to export the path to where 
applications install to.. (/usr/bin? /Applications? C:\Program Files?) I guess 
it really depends.

-- 
Thomas Fjellstrom
tfjellstrom@xxxxxxxxxx




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