From 2daa4db8a0cb23dfd0ca417ead2413e20ec18fe7 Mon Sep 17 00:00:00 2001 From: Bastien Montagne Date: Wed, 15 Jul 2015 17:43:04 +0200 Subject: [PATCH] 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... --- source/blender/blenlib/intern/fileops.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) 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;