Cleanup: rename natural string comparison
authorCampbell Barton <ideasman42@gmail.com>
Sat, 31 Aug 2019 07:28:48 +0000 (17:28 +1000)
committerCampbell Barton <ideasman42@gmail.com>
Sat, 31 Aug 2019 07:32:30 +0000 (17:32 +1000)
source/blender/blenlib/BLI_string.h
source/blender/blenlib/intern/BLI_filelist.c
source/blender/blenlib/intern/string.c
source/blender/collada/AnimationImporter.cpp
source/blender/editors/object/object_vgroup.c
source/blender/editors/space_file/filelist.c
source/blender/editors/space_node/node_templates.c
source/blender/editors/space_outliner/outliner_tree.c

index 70ffb46e952aa1739a6d179ee21dea29db2e6458..cab2e68ca2b91aa681b9927939f3e270ea391ee0 100644 (file)
@@ -100,7 +100,7 @@ char *BLI_strncasestr(const char *s, const char *find, size_t len) ATTR_WARN_UNU
 int BLI_strcasecmp(const char *s1, const char *s2) ATTR_WARN_UNUSED_RESULT ATTR_NONNULL();
 int BLI_strncasecmp(const char *s1, const char *s2, size_t len) ATTR_WARN_UNUSED_RESULT
     ATTR_NONNULL();
-int BLI_natstrcmp(const char *s1, const char *s2) ATTR_WARN_UNUSED_RESULT ATTR_NONNULL();
+int BLI_strcasecmp_natural(const char *s1, const char *s2) ATTR_WARN_UNUSED_RESULT ATTR_NONNULL();
 int BLI_strcmp_ignore_pad(const char *str1,
                           const char *str2,
                           const char pad) ATTR_WARN_UNUSED_RESULT ATTR_NONNULL();
index 91f16ca9b7b3cd5c791ee96dee18519cc999c5e2..a93632cca08fccd862ba606279038854f93fd89f 100644 (file)
@@ -107,7 +107,7 @@ static int bli_compare(struct direntry *entry1, struct direntry *entry2)
     return 1;
   }
 
-  return (BLI_natstrcmp(entry1->relname, entry2->relname));
+  return (BLI_strcasecmp_natural(entry1->relname, entry2->relname));
 }
 
 struct BuildDirCtx {
index c3947cb8158f7bbc54f11cb3164e11f7a81ee42c..4059e0401c068229fc91d8bd8343a79ac83367f0 100644 (file)
@@ -663,8 +663,11 @@ static int left_number_strcmp(const char *s1, const char *s2, int *tiebreaker)
   return 0;
 }
 
-/* natural string compare, keeping numbers in order */
-int BLI_natstrcmp(const char *s1, const char *s2)
+/**
+ * Case insensitive, *natural* string comparison,
+ * keeping numbers in order.
+ */
+int BLI_strcasecmp_natural(const char *s1, const char *s2)
 {
   register int d1 = 0, d2 = 0;
   register char c1, c2;
index 54177560eb5010fe7e978c305e92a1831fe514e6..79593f07383c280b63e880456f5cde8abec6f613 100644 (file)
@@ -733,7 +733,7 @@ void AnimationImporter::Assign_float_animations(const COLLADAFW::UniqueId &listi
            * Reason: old blender versions stored spot_size in radians (was a bug)
            */
           if (this->import_from_version == "" ||
-              BLI_natstrcmp(this->import_from_version.c_str(), "2.69.10") != -1) {
+              BLI_strcasecmp_natural(this->import_from_version.c_str(), "2.69.10") != -1) {
             fcurve_deg_to_rad(fcu);
           }
         }
index 08fe5e818b2d95384ee2edbf8e32dc4fd4d9abaa..ce385b12608ab63604e96ae5f1a4c1c9a5f953ae 100644 (file)
@@ -3794,7 +3794,7 @@ static int vgroup_sort_name(const void *def_a_ptr, const void *def_b_ptr)
   const bDeformGroup *def_a = def_a_ptr;
   const bDeformGroup *def_b = def_b_ptr;
 
-  return BLI_natstrcmp(def_a->name, def_b->name);
+  return BLI_strcasecmp_natural(def_a->name, def_b->name);
 }
 
 /**
index f7dda1defe82fbe01b0a4b7672a31e28182c2211..ee86a5839749997ec38ff4040c8de2c0e629c06a 100644 (file)
@@ -434,7 +434,7 @@ static int compare_name(void *UNUSED(user_data), const void *a1, const void *a2)
   name1 = entry1->name;
   name2 = entry2->name;
 
-  return BLI_natstrcmp(name1, name2);
+  return BLI_strcasecmp_natural(name1, name2);
 }
 
 static int compare_date(void *UNUSED(user_data), const void *a1, const void *a2)
@@ -461,7 +461,7 @@ static int compare_date(void *UNUSED(user_data), const void *a1, const void *a2)
   name1 = entry1->name;
   name2 = entry2->name;
 
-  return BLI_natstrcmp(name1, name2);
+  return BLI_strcasecmp_natural(name1, name2);
 }
 
 static int compare_size(void *UNUSED(user_data), const void *a1, const void *a2)
@@ -488,7 +488,7 @@ static int compare_size(void *UNUSED(user_data), const void *a1, const void *a2)
   name1 = entry1->name;
   name2 = entry2->name;
 
-  return BLI_natstrcmp(name1, name2);
+  return BLI_strcasecmp_natural(name1, name2);
 }
 
 static int compare_extension(void *UNUSED(user_data), const void *a1, const void *a2)
@@ -546,7 +546,7 @@ static int compare_extension(void *UNUSED(user_data), const void *a1, const void
   name1 = entry1->name;
   name2 = entry2->name;
 
-  return BLI_natstrcmp(name1, name2);
+  return BLI_strcasecmp_natural(name1, name2);
 }
 
 void filelist_sort(struct FileList *filelist)
index 8cc57a82fe0e519fbe7c1b05ca329444fc592049..423dec13c69b5640b47c734163b7ec2fb6c94106 100644 (file)
@@ -441,7 +441,7 @@ static int ui_node_item_name_compare(const void *a, const void *b)
 {
   const bNodeType *type_a = *(const bNodeType **)a;
   const bNodeType *type_b = *(const bNodeType **)b;
-  return BLI_natstrcmp(type_a->ui_name, type_b->ui_name);
+  return BLI_strcasecmp_natural(type_a->ui_name, type_b->ui_name);
 }
 
 static bool ui_node_item_special_poll(const bNodeTree *UNUSED(ntree), const bNodeType *ntype)
index 4195865668849909861cf5c13b0632aaa861d3d3..fd6a052b84d7e2f6ccd20f860bdce554115adca8 100644 (file)
@@ -1627,7 +1627,7 @@ static int treesort_alpha_ob(const void *v1, const void *v2)
       return (x1->te->flag & TE_CHILD_NOT_IN_COLLECTION) ? 1 : -1;
     }
 
-    comp = BLI_natstrcmp(x1->name, x2->name);
+    comp = BLI_strcasecmp_natural(x1->name, x2->name);
 
     if (comp > 0) {
       return 1;
@@ -1659,7 +1659,7 @@ static int treesort_alpha(const void *v1, const void *v2)
   const tTreeSort *x1 = v1, *x2 = v2;
   int comp;
 
-  comp = BLI_natstrcmp(x1->name, x2->name);
+  comp = BLI_strcasecmp_natural(x1->name, x2->name);
 
   if (comp > 0) {
     return 1;
@@ -1697,7 +1697,7 @@ static int treesort_obtype_alpha(const void *v1, const void *v2)
       }
     }
     else {
-      int comp = BLI_natstrcmp(x1->name, x2->name);
+      int comp = BLI_strcasecmp_natural(x1->name, x2->name);
 
       if (comp > 0) {
         return 1;