Re: [AD] al_make_recursive_directory

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


On Sun, 2010-10-24 at 10:19 +1100, Peter Wang wrote:
> > 
> > Here's the real patch.
> 
> Actually this is the real real patch.

Well, shows that we need the function in any case, much less trivial
than it seemed :)

> The reason I think the logic should be pushed into the fshook
> implementations is that some file systems may not recognise '/' as
> directory separators

Hm, ALLEGRO_NATIVE_PATH_SEP should always work though, else we have
another usability problem.

>  and they may not *need* parent directories to be
> created separately.
> 

I see. Well, right now the only place it is needed is in fshooks.c
anyway - the physfs version already creates all parent directories if
they are missing [1].

[1] http://icculus.org/physfs/docs/html/physfs_8h.html#ae11fb98bf8c08a2e028f52ac9a728aa9

-- 
Elias Pschernig <elias.pschernig@xxxxxxxxxx>





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