[vlc-devel] commit: dvdnav: Remove trailing backslash on Win32 ( Marian Ďurkovič )

Marian Ďurkovič md at bts.sk
Mon Oct 19 11:18:22 CEST 2009


On Mon, Oct 19, 2009 at 10:01:56AM +0200, Rémi Denis-Courmont wrote:
> AFAICT, you can remove the trailing path separator on any platform.

Is it necessary (i.e. it doesn't work without it on any other platform)
or just optional?

If optional, I'd suggest not to mess with it, as there might be some
corner cases - e.g.  "/" changed to "". The fact that libdvdnav doesn't
accept trailing backslash on win32 seems more like a bug to me, as
cd c:\windows\    definitely works on win32 systems. 

BTW, libdvdnav still seems to use the ANSI functions and not Unicode ones
on Win32 :-(

   With kind regards,

        M.



More information about the vlc-devel mailing list