Fix filebrowser not getting back to valid dir in Release builds.
authorBastien Montagne <montagne29@wanadoo.fr>
Thu, 15 Sep 2016 10:12:19 +0000 (12:12 +0200)
committerSergey Sharybin <sergey.vfx@gmail.com>
Thu, 22 Sep 2016 09:07:35 +0000 (11:07 +0200)
Stupid mistake wrapping path validation code inside a BLI_assert, which means it was
only called in Debug builds...

Found by Sergey, thanks.

Should be backported to 2.78.

source/blender/editors/space_file/filelist.c

index 1471932..ab95a77 100644 (file)
@@ -1405,7 +1405,9 @@ void filelist_setdir(struct FileList *filelist, char *r_dir)
        BLI_assert(strlen(r_dir) < FILE_MAX_LIBEXTRA);
 
        BLI_cleanup_dir(G.main->name, r_dir);
-       BLI_assert(filelist->checkdirf(filelist, r_dir, true));
+       const bool is_valid_path = filelist->checkdirf(filelist, r_dir, true);
+       BLI_assert(is_valid_path);
+       UNUSED_VARS_NDEBUG(is_valid_path);
 
        if (!STREQ(filelist->filelist.root, r_dir)) {
                BLI_strncpy(filelist->filelist.root, r_dir, sizeof(filelist->filelist.root));