Re: [AD] SF.net SVN: alleg:[13860] allegro/branches/4.9

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


On October 30, 2010, tjaden@xxxxxxxxxx wrote:
> Revision: 13860
>           http://alleg.svn.sourceforge.net/alleg/?rev=13860&view=rev
> Author:   tjaden
> Date:     2010-10-30 07:46:26 +0000 (Sat, 30 Oct 2010)
> 
> Log Message:
> -----------
> Make al_get_fs_entry_name and al_get_current_directory return strings
> instead of ALLEGRO_PATH.  The filesystem functions all work with strings
> except for those two.  ALLEGRO_PATHs may not be suitable for some
> filesystems which use a different path syntax.
> 
> There is another link between the path and filesystem subsystems which
> may be unintended.  al_make_path_absolute depends on the current
> filesystem in effect.
> 

Most API's I've played with depend on the "make absolute" function using the 
current cwd state. So I don't really think its a problem as is.

That said, I guess it might make more sense to pass the base path into 
al_make_path_absolute, or just remove it and say "use al_join_paths".. 

-- 
Thomas Fjellstrom
tfjellstrom@xxxxxxxxxx




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