diff options
author | Bastien Montagne <montagne29@wanadoo.fr> | 2015-07-15 18:43:04 +0300 |
---|---|---|
committer | Bastien Montagne <montagne29@wanadoo.fr> | 2015-07-15 18:43:04 +0300 |
commit | 2daa4db8a0cb23dfd0ca417ead2413e20ec18fe7 (patch) | |
tree | 906a8498a09782bf7e158bc7d460362d94f3ed9d /source/blender/blenlib | |
parent | 548e6502524f7f3ed068ee073c5d0728eb76e9a6 (diff) |
Fix own stupid error in yesterday's refactor of recursive dir creation.
Missed adding return value in one place.
Thanks a lot to Thomas Szepe for reporting the issue and finding the solution!
Question remains: how could MSVC allow such a code to compile in some cases
(own win64 + scons buildbot win32 were OK with that it seems!).
Crappy compiler, not comming to the rescue of stupid dev...
Diffstat (limited to 'source/blender/blenlib')
-rw-r--r-- | source/blender/blenlib/intern/fileops.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/source/blender/blenlib/intern/fileops.c b/source/blender/blenlib/intern/fileops.c index 5232c99944a..bf9daea7f7c 100644 --- a/source/blender/blenlib/intern/fileops.c +++ b/source/blender/blenlib/intern/fileops.c @@ -448,7 +448,9 @@ bool BLI_dir_create_recursive(const char *dirname) BLI_del_slash(tmp); /* check special case "c:\foo", don't try create "c:", harmless but prints an error below */ - if (isalpha(tmp[0]) && (tmp[1] == ':') && tmp[2] == '\0') return; + if (isalpha(tmp[0]) && (tmp[1] == ':') && tmp[2] == '\0') { + return true; + } if (BLI_is_dir(tmp)) { return true; |