Revision: 75941
          http://sourceforge.net/p/brlcad/code/75941
Author:   starseeker
Date:     2020-05-26 20:53:53 +0000 (Tue, 26 May 2020)
Log Message:
-----------
simplify

Modified Paths:
--------------
    brlcad/trunk/src/libged/which.cpp

Added Paths:
-----------
    brlcad/trunk/src/libged/alphanum.hpp

Removed Paths:
-------------
    brlcad/trunk/src/libged/alphanum.h

Deleted: brlcad/trunk/src/libged/alphanum.h
===================================================================
--- brlcad/trunk/src/libged/alphanum.h  2020-05-26 19:48:25 UTC (rev 75940)
+++ brlcad/trunk/src/libged/alphanum.h  2020-05-26 20:53:53 UTC (rev 75941)
@@ -1,284 +0,0 @@
-#ifndef ALPHANUM__HPP
-#define ALPHANUM__HPP
-
-/*
-The Alphanum Algorithm is an improved sorting algorithm for strings
-containing numbers.  Instead of sorting numbers in ASCII order like a
-standard sort, this algorithm sorts numbers in numeric order.
-
-The Alphanum Algorithm is discussed at http://www.DaveKoelle.com
-
-This implementation is Copyright (c) 2008 Dirk Jagdmann <[email protected]>.
-It is a cleanroom implementation of the algorithm and not derived by
-other's works. In contrast to the versions written by Dave Koelle this
-source code is distributed with the libpng/zlib license.
-
-This software is provided 'as-is', without any express or implied
-warranty. In no event will the authors be held liable for any damages
-arising from the use of this software.
-
-Permission is granted to anyone to use this software for any purpose,
-including commercial applications, and to alter it and redistribute it
-freely, subject to the following restrictions:
-
-    1. The origin of this software must not be misrepresented; you
-       must not claim that you wrote the original software. If you use
-       this software in a product, an acknowledgment in the product
-       documentation would be appreciated but is not required.
-
-    2. Altered source versions must be plainly marked as such, and
-       must not be misrepresented as being the original software.
-
-    3. This notice may not be removed or altered from any source
-       distribution.
-
-Source:
-https://github.com/facontidavide/PlotJuggler/blob/master/include/PlotJuggler/alphanum.hpp
-*/
-
-#include <cassert>
-#include <functional>
-#include <string>
-#include <sstream>
-
-#define ALPHANUM_LOCALE
-
-#ifdef ALPHANUM_LOCALE
-#include <cctype>
-#endif
-
-namespace doj
-{
-
-  // anonymous namespace for functions we use internally. But if you
-  // are coding in C, you can use alphanum_impl() directly, since it
-  // uses not C++ features.
-  namespace {
-
-    // if you want to honour the locale settings for detecting digit
-    // characters, you should define ALPHANUM_LOCALE
-#ifdef ALPHANUM_LOCALE
-    /** wrapper function for ::isdigit() */
-    bool alphanum_isdigit(int c)
-    {
-      return isdigit(c);
-    }
-#else
-    /** this function does not consider the current locale and only
-       works with ASCII digits.
-       @return true if c is a digit character
-    */
-    bool alphanum_isdigit(const char c)
-    {
-      return c>='0' && c<='9';
-    }
-#endif
-
-    /**
-       compare l and r with strcmp() semantics, but using
-       the "Alphanum Algorithm". This function is designed to read
-       through the l and r strings only one time, for
-       maximum performance. It does not allocate memory for
-       substrings. It can either use the C-library functions isdigit()
-       and atoi() to honour your locale settings, when recognizing
-       digit characters when you "#define ALPHANUM_LOCALE=1" or use
-       it's own digit character handling which only works with ASCII
-       digit characters, but provides better performance.
-
-       @param l NULL-terminated C-style string
-       @param r NULL-terminated C-style string
-       @return negative if l<r, 0 if l equals r, positive if l>r
-     */
-    int alphanum_impl(const char *l, const char *r)
-    {
-      enum mode_t { STRING, NUMBER } mode=STRING;
-
-      while(*l && *r)
-       {
-         if(mode == STRING)
-           {
-             char l_char, r_char;
-             while((l_char=*l) && (r_char=*r))
-               {
-                 // check if this are digit characters
-                 const bool l_digit=alphanum_isdigit(l_char), 
r_digit=alphanum_isdigit(r_char);
-                 // if both characters are digits, we continue in NUMBER mode
-                 if(l_digit && r_digit)
-                   {
-                     mode=NUMBER;
-                     break;
-                   }
-                 // if only the left character is a digit, we have a result
-                 if(l_digit) return -1;
-                 // if only the right character is a digit, we have a result
-                 if(r_digit) return +1;
-                 // compute the difference of both characters
-                 const int diff=l_char - r_char;
-                 // if they differ we have a result
-                 if(diff != 0) return diff;
-                 // otherwise process the next characters
-                 ++l;
-                 ++r;
-               }
-           }
-         else // mode==NUMBER
-           {
-#ifdef ALPHANUM_LOCALE
-             // get the left number
-             char *end;
-             unsigned long l_int=strtoul(l, &end, 0);
-             l=end;
-
-             // get the right number
-             unsigned long r_int=strtoul(r, &end, 0);
-             r=end;
-#else
-             // get the left number
-             unsigned long l_int=0;
-             while(*l && alphanum_isdigit(*l))
-               {
-                 // TODO: this can overflow
-                 l_int=l_int*10 + *l-'0';
-                 ++l;
-               }
-
-             // get the right number
-             unsigned long r_int=0;
-             while(*r && alphanum_isdigit(*r))
-               {
-                 // TODO: this can overflow
-                 r_int=r_int*10 + *r-'0';
-                 ++r;
-               }
-#endif
-
-             // if the difference is not equal to zero, we have a comparison 
result
-             const long diff=l_int-r_int;
-             if(diff != 0)
-               return diff;
-
-             // otherwise we process the next substring in STRING mode
-             mode=STRING;
-           }
-       }
-
-      if(*r) return -1;
-      if(*l) return +1;
-      return 0;
-    }
-
-  }
-
-  /**
-     Compare left and right with the same semantics as strcmp(), but with the
-     "Alphanum Algorithm" which produces more human-friendly
-     results. The classes lT and rT must implement "std::ostream
-     operator<< (std::ostream&, const Ty&)".
-
-     @return negative if left<right, 0 if left==right, positive if left>right.
-  */
-  template <typename lT, typename rT>
-  int alphanum_comp(const lT& left, const rT& right)
-  {
-    std::ostringstream l; l << left;
-    std::ostringstream r; r << right;
-    return alphanum_impl(l.str().c_str(), r.str().c_str());
-  }
-
-  /**
-     Compare l and r with the same semantics as strcmp(), but with
-     the "Alphanum Algorithm" which produces more human-friendly
-     results.
-
-     @return negative if l<r, 0 if l==r, positive if l>r.
-  */
-  template <> inline
-  int alphanum_comp<std::string>(const std::string& l, const std::string& r)
-  {
-    return alphanum_impl(l.c_str(), r.c_str());
-  }
-
-  ////////////////////////////////////////////////////////////////////////////
-
-  // now follow a lot of overloaded alphanum_comp() functions to get a
-  // direct call to alphanum_impl() upon the various combinations of c
-  // and c++ strings.
-
-  /**
-     Compare l and r with the same semantics as strcmp(), but with
-     the "Alphanum Algorithm" which produces more human-friendly
-     results.
-
-     @return negative if l<r, 0 if l==r, positive if l>r.
-  */
-  inline int alphanum_comp(char* l, char* r)
-  {
-    assert(l);
-    assert(r);
-    return alphanum_impl(l, r);
-  }
-
-  inline int alphanum_comp(const char* l, const char* r)
-  {
-    assert(l);
-    assert(r);
-    return alphanum_impl(l, r);
-  }
-
-  inline int alphanum_comp(char* l, const char* r)
-  {
-    assert(l);
-    assert(r);
-    return alphanum_impl(l, r);
-  }
-
-  inline int alphanum_comp(const char* l, char* r)
-  {
-    assert(l);
-    assert(r);
-    return alphanum_impl(l, r);
-  }
-
-  inline int alphanum_comp(const std::string& l, char* r)
-  {
-    assert(r);
-    return alphanum_impl(l.c_str(), r);
-  }
-
-  inline int alphanum_comp(char* l, const std::string& r)
-  {
-    assert(l);
-    return alphanum_impl(l, r.c_str());
-  }
-
-  inline int alphanum_comp(const std::string& l, const char* r)
-  {
-    assert(r);
-    return alphanum_impl(l.c_str(), r);
-  }
-
-  inline int alphanum_comp(const char* l, const std::string& r)
-  {
-    assert(l);
-    return alphanum_impl(l, r.c_str());
-  }
-
-  ////////////////////////////////////////////////////////////////////////////
-
-  /**
-     Functor class to compare two objects with the "Alphanum
-     Algorithm". If the objects are no std::string, they must
-     implement "std::ostream operator<< (std::ostream&, const Ty&)".
-  */
-  template<class Ty>
-  struct alphanum_less : public std::binary_function<Ty, Ty, bool>
-  {
-    bool operator()(const Ty& left, const Ty& right) const
-    {
-      return alphanum_comp(left, right) < 0;
-    }
-  };
-
-}
-
-#endif

Copied: brlcad/trunk/src/libged/alphanum.hpp (from rev 75940, 
brlcad/trunk/src/libged/alphanum.h)
===================================================================
--- brlcad/trunk/src/libged/alphanum.hpp                                (rev 0)
+++ brlcad/trunk/src/libged/alphanum.hpp        2020-05-26 20:53:53 UTC (rev 
75941)
@@ -0,0 +1,134 @@
+#ifndef ALPHANUM__HPP
+#define ALPHANUM__HPP
+
+/*
+   The Alphanum Algorithm is an improved sorting algorithm for strings
+   containing numbers.  Instead of sorting numbers in ASCII order like a
+   standard sort, this algorithm sorts numbers in numeric order.
+
+   The Alphanum Algorithm is discussed at http://www.DaveKoelle.com
+
+   This implementation is Copyright (c) 2008 Dirk Jagdmann <[email protected]>.
+   It is a cleanroom implementation of the algorithm and not derived by
+   other's works. In contrast to the versions written by Dave Koelle this
+   source code is distributed with the libpng/zlib license.
+
+   This software is provided 'as-is', without any express or implied
+   warranty. In no event will the authors be held liable for any damages
+   arising from the use of this software.
+
+   Permission is granted to anyone to use this software for any purpose,
+   including commercial applications, and to alter it and redistribute it
+   freely, subject to the following restrictions:
+
+   1. The origin of this software must not be misrepresented; you
+   must not claim that you wrote the original software. If you use
+   this software in a product, an acknowledgment in the product
+   documentation would be appreciated but is not required.
+
+   2. Altered source versions must be plainly marked as such, and
+   must not be misrepresented as being the original software.
+
+   3. This notice may not be removed or altered from any source
+   distribution.
+
+Source:
+https://github.com/facontidavide/PlotJuggler/blob/master/include/PlotJuggler/alphanum.hpp
+
+Simplified down to the essential sorting comparison of C strings 
+
+*/
+
+#include "common.h"
+
+#include <cassert>
+#include <functional>
+#include <string>
+#include <sstream>
+#include <cctype>
+
+/**
+  compare l and r with strcmp() semantics, but using
+  the "Alphanum Algorithm". This function is designed to read
+  through the l and r strings only one time, for
+  maximum performance. It does not allocate memory for
+  substrings. It can either use the C-library functions isdigit()
+  and atoi() to honour your locale settings, when recognizing
+  digit characters when you "#define ALPHANUM_LOCALE=1" or use
+  it's own digit character handling which only works with ASCII
+  digit characters, but provides better performance.
+
+  @param l NULL-terminated C-style string
+  @param r NULL-terminated C-style string
+  @return negative if l<r, 0 if l equals r, positive if l>r
+  */
+int alphanum_impl(const char *l, const char *r, void *UNUSED(arg))
+{
+    enum alphanum_mode_t { STRING, NUMBER } mode=STRING;
+
+    while(*l && *r)
+    {
+       if(mode == STRING)
+       {
+           char l_char, r_char;
+           while((l_char=*l) && (r_char=*r))
+           {
+               // check if this are digit characters
+               const bool l_digit=isdigit(l_char), r_digit=isdigit(r_char);
+               // if both characters are digits, we continue in NUMBER mode
+               if(l_digit && r_digit)
+               {
+                   mode=NUMBER;
+                   break;
+               }
+               // if only the left character is a digit, we have a result
+               if(l_digit) return -1;
+               // if only the right character is a digit, we have a result
+               if(r_digit) return +1;
+               // compute the difference of both characters
+               const int diff=l_char - r_char;
+               // if they differ we have a result
+               if(diff != 0) return diff;
+               // otherwise process the next characters
+               ++l;
+               ++r;
+           }
+       }
+       else // mode==NUMBER
+       {
+           // get the left number
+           char *end;
+           unsigned long l_int=strtoul(l, &end, 0);
+           l=end;
+
+           // get the right number
+           unsigned long r_int=strtoul(r, &end, 0);
+           r=end;
+
+           // if the difference is not equal to zero, we have a comparison 
result
+           const long diff=l_int-r_int;
+           if(diff != 0)
+               return diff;
+
+           // otherwise we process the next substring in STRING mode
+           mode=STRING;
+       }
+    }
+
+    if(*r) return -1;
+    if(*l) return +1;
+    return 0;
+}
+
+
+#endif
+
+
+// Local Variables:
+// tab-width: 8
+// mode: C++
+// c-basic-offset: 4
+// indent-tabs-mode: t
+// c-file-style: "stroustrup"
+// End:
+// ex: shiftwidth=4 tabstop=8

Modified: brlcad/trunk/src/libged/which.cpp
===================================================================
--- brlcad/trunk/src/libged/which.cpp   2020-05-26 19:48:25 UTC (rev 75940)
+++ brlcad/trunk/src/libged/which.cpp   2020-05-26 20:53:53 UTC (rev 75941)
@@ -34,11 +34,11 @@
 #include "bu/opt.h"
 #include "bu/vls.h"
 
-#include "./alphanum.h"
+#include "./alphanum.hpp"
 #include "./ged_private.h"
 
 bool alphanum_cmp(const std::string& a, const std::string& b) {
-    return doj::alphanum_impl(a.c_str(), b.c_str()) < 0;
+    return alphanum_impl(a.c_str(), b.c_str(), NULL) < 0;
 }
 
 int

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.



_______________________________________________
BRL-CAD Source Commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-commits

Reply via email to