Am Montag, 3. Oktober 2016, 20:56:39 CEST schrieb Jan Ingwer Baer:
> Hi,

Hi.

[...]

> - In common/film.c function film_recursive_get_files() there is a
> mistake in the call to function g_build_filename(). It is called with
> the same parameters as g_build_pathname(), but according to the glib
> manual it does not need the first parameter (G_DIR_SEPARATOR_S). Under
> windows this will produce an invalid filename with a leading slash (like
> /c:/user/...). Under *ix this no problem because it will simple double
> the leading slash of an absolute path what the os ignores.

Fixed in master.

> - In common/darktable.c function strip_semicolons_from_keymap() there is
> only a test for the result of fgetc() !=eof. But on mingw this doesnt
> work, and it must test for feof() ==0. Without the test for feof() the
> result is an infinite loop...

Then your implementation of fgetc is broken. It is required to return EOF when 
the end of the file is reached. Maybe it's a problem with line endings?

[...]

> Greetings from Berlin
> 
> Jan Ingwer Baer

Thanks for reporting the bug.

Tobias

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to