Commit: 2daa4db8a0cb23dfd0ca417ead2413e20ec18fe7
Author: Bastien Montagne
Date:   Wed Jul 15 17:43:04 2015 +0200
Branches: master
https://developer.blender.org/rB2daa4db8a0cb23dfd0ca417ead2413e20ec18fe7

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...

===================================================================

M       source/blender/blenlib/intern/fileops.c

===================================================================

diff --git a/source/blender/blenlib/intern/fileops.c 
b/source/blender/blenlib/intern/fileops.c
index 5232c99..bf9daea 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;

_______________________________________________
Bf-blender-cvs mailing list
[email protected]
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to