diff options
author | Campbell Barton <ideasman42@gmail.com> | 2012-12-24 18:59:15 +0400 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2012-12-24 18:59:15 +0400 |
commit | fb989c2a6531ffb53aad2cb8aba85f974d8e9fcf (patch) | |
tree | 155d2f726ccc96ae7cbd52872b89ff02b1496bd4 /source/blender/blenlib | |
parent | 128e6d51baebb9e3724cf84538ec074ca3f230b3 (diff) |
code cleanup: don't alloca zero size and remove paranoid NULL checks (checked all uses and there not needed).
Diffstat (limited to 'source/blender/blenlib')
-rw-r--r-- | source/blender/blenlib/BLI_path_util.h | 12 | ||||
-rw-r--r-- | source/blender/blenlib/intern/path_util.c | 3 |
2 files changed, 10 insertions, 5 deletions
diff --git a/source/blender/blenlib/BLI_path_util.h b/source/blender/blenlib/BLI_path_util.h index 557ecb3dd0c..8c51c925d97 100644 --- a/source/blender/blenlib/BLI_path_util.h +++ b/source/blender/blenlib/BLI_path_util.h @@ -120,7 +120,11 @@ int BLI_split_name_num(char *left, int *nr, const char *name, const char delim); void BLI_splitdirstring(char *di, char *fi); /* make sure path separators conform to system one */ -void BLI_clean(char *path); +void BLI_clean(char *path) +#ifdef __GNUC__ +__attribute__((nonnull(1))) +#endif +; /** * dir can be any input, like from buttons, and this function @@ -173,7 +177,11 @@ int BLI_path_is_rel(const char *path); * \a from The character to replace * \a to The character to replace with */ -void BLI_char_switch(char *string, char from, char to); +void BLI_char_switch(char *string, char from, char to) +#ifdef __GNUC__ +__attribute__((nonnull(1))) +#endif +; /* Initialize path to program executable */ void BLI_init_program_path(const char *argv0); diff --git a/source/blender/blenlib/intern/path_util.c b/source/blender/blenlib/intern/path_util.c index 7c1842b10f2..3c43fe2fb72 100644 --- a/source/blender/blenlib/intern/path_util.c +++ b/source/blender/blenlib/intern/path_util.c @@ -1212,8 +1212,6 @@ void BLI_setenv_if_new(const char *env, const char *val) void BLI_clean(char *path) { - if (path == NULL) return; - #ifdef WIN32 if (path && BLI_strnlen(path, 3) > 2) { BLI_char_switch(path + 2, '/', '\\'); @@ -1225,7 +1223,6 @@ void BLI_clean(char *path) void BLI_char_switch(char *string, char from, char to) { - if (string == NULL) return; while (*string != 0) { if (*string == from) *string = to; string++; |