Cleanup: return early in BLI_path_frame_get
authorSybren A. Stüvel <sybren@stuvel.eu>
Wed, 20 Mar 2019 12:39:27 +0000 (13:39 +0100)
committerSybren A. Stüvel <sybren@stuvel.eu>
Wed, 20 Mar 2019 12:45:14 +0000 (13:45 +0100)
Instead of making the entire body of the function conditional, it now
returns early, unindenting the entire function and preventing the reader
from searching for a non-existent `else` clause.

No semantic changes.

source/blender/blenlib/intern/path_util.c

index 6d69f29..34fb711 100644 (file)
@@ -864,47 +864,49 @@ bool BLI_path_frame_get(char *path, int *r_frame, int *r_numdigits)
 
 void BLI_path_frame_strip(char *path, char *r_ext)
 {
-       if (*path) {
-               char *file = (char *)BLI_last_slash(path);
-               char *c, *suffix;
-               int len;
-               int numdigits = 0;
-
-               if (file == NULL)
-                       file = path;
+       if (*path == '\0') {
+               return;
+       }
 
-               /* first get the extension part */
-               len = strlen(file);
+       char *file = (char *)BLI_last_slash(path);
+       char *c, *suffix;
+       int len;
+       int numdigits = 0;
 
-               c = file + len;
+       if (file == NULL)
+               file = path;
 
-               /* isolate extension */
-               while (--c != file) {
-                       if (*c == '.') {
-                               c--;
-                               break;
-                       }
-               }
+       /* first get the extension part */
+       len = strlen(file);
 
-               suffix = c + 1;
+       c = file + len;
 
-               /* find start of number */
-               while (c != (file - 1) && isdigit(*c)) {
+       /* isolate extension */
+       while (--c != file) {
+               if (*c == '.') {
                        c--;
-                       numdigits++;
+                       break;
                }
+       }
 
-               c++;
+       suffix = c + 1;
 
-               int suffix_length = len - (suffix - file);
-               BLI_strncpy(r_ext, suffix, suffix_length+1);
+       /* find start of number */
+       while (c != (file - 1) && isdigit(*c)) {
+               c--;
+               numdigits++;
+       }
 
-               /* replace the number with the suffix and terminate the string */
-               while (numdigits--) {
-                       *c++ = '#';
-               }
-               *c = '\0';
+       c++;
+
+       int suffix_length = len - (suffix - file);
+       BLI_strncpy(r_ext, suffix, suffix_length+1);
+
+       /* replace the number with the suffix and terminate the string */
+       while (numdigits--) {
+               *c++ = '#';
        }
+       *c = '\0';
 }