On 17 December 2013 18:40, Thiago Macieira <[email protected]> wrote: > On terça-feira, 17 de dezembro de 2013 06:48:51, Dirk Hohndel wrote: >> Thiago, can I get an ACK from you for this patch, please? > > ACK, but incomplete. > > There are other places where we encode filenames to 8-bit. We should: > > - ensure that all file names are encoded with QFile::encodeName and decoded > with QFile::decodeName > - we initialise QFile on Windows (setEncodingFunction, setDecodingFunction) > - fix other places where we open files, including zip_open[1] > > I can do the first two. I see other open calls in save-xml.c and uemis- > downloader.c, plus the zip_open calls and one sqlite3_open. > > [1] We need to fix libzip upstream with either an encoding mechanism for > windows or a zip_wopen. >
in that case, i would discard my patch for now. i can fix step 3, with wrappers in windows.c, linux.c, etc... and basically provide the following wrappers (OTTOMH): subsurface_open() subsurface_fopen() subsurface_zip_open() can't seem to find zip_wopen in their API, which means we need to use _wopen with zip_fdopen if someone has better ideas please, or corrections please share them. i probably start making these changes tomorrow morning GMT. lubomir -- _______________________________________________ subsurface mailing list [email protected] http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface
