home *** CD-ROM | disk | FTP | other *** search
/ vim.ftp.fu-berlin.de / 2015-02-03.vim.ftp.fu-berlin.de.tar / vim.ftp.fu-berlin.de / patches / old / 5.4.36 < prev    next >
Encoding:
Internet Message Format  |  1999-08-18  |  4.6 KB
  1. To: Mike Steed <mrsteed@usa.net>
  2. In-Reply-To: <19990818231828.6484.qmail@www0j.netaddress.usa.net>
  3. Cc: vim-dev@vim.org
  4. Subject: patch 5.4.36 (was: [Comparing strings?])
  5. Fcc: outbox
  6. From: Bram Moolenaar <Bram@moolenaar.net>
  7. ------------
  8.  
  9. Mike Steed wrote:
  10.  
  11. > Okay, I think I found the problem.  My build of Vim had
  12. > 
  13. >    #define STRICMP(d, s)   vim_stricmp((char *)(d), (char *)(s))
  14. > 
  15. > And it turns out that vim_stricmp() returns 0 when d == s, and 1 when d != s. 
  16. > Shouldn't this routine be fixed to behave like strcasecmp() -- to return -1,
  17. > 0, or 1, telling whether d < s, d == s, or d > s?
  18.  
  19. Yes, this needs to be fixed.  But I don't understand why your build was using
  20. vim_stricmp().  On Win32 the library stricmp() should have been used.  In
  21. fact, most machines would not use this.
  22.  
  23.  
  24. Patch 5.4.36
  25. Problem:    Comparing strings while ignoring case didn't work correctly for
  26.         some machines. (Mide Steed)
  27. Solution:   vim_stricmp() and vim_strnicmp() only returned 0 or 1.  Changed
  28.         them to return -1 when the first argument is smaller.
  29. Files:        src/misc2.c
  30.  
  31.  
  32. *** ../vim-5.4.35/src/misc2.c    Fri Jul  2 21:59:18 1999
  33. --- src/misc2.c    Thu Aug 19 12:03:13 1999
  34. ***************
  35. *** 954,970 ****
  36.   #if (!defined(HAVE_STRCASECMP) && !defined(HAVE_STRICMP)) || defined(PROTO)
  37.   /*
  38.    * Compare two strings, ignoring case.
  39. !  * return 0 for match, 1 for difference
  40.    */
  41.       int
  42.   vim_stricmp(s1, s2)
  43. !     char    *s1;
  44. !     char    *s2;
  45.   {
  46.       for (;;)
  47.       {
  48. !     if (TO_LOWER(*s1) != TO_LOWER(*s2))
  49. !         return 1;                /* this character different */
  50.       if (*s1 == NUL)
  51.           break;                /* strings match until NUL */
  52.       ++s1;
  53. --- 954,973 ----
  54.   #if (!defined(HAVE_STRCASECMP) && !defined(HAVE_STRICMP)) || defined(PROTO)
  55.   /*
  56.    * Compare two strings, ignoring case.
  57. !  * return 0 for match, < 0 for smaller, > 0 for bigger
  58.    */
  59.       int
  60.   vim_stricmp(s1, s2)
  61. !     char    *s1;
  62. !     char    *s2;
  63.   {
  64. +     int        i;
  65. + 
  66.       for (;;)
  67.       {
  68. !     i = (int)TO_LOWER(*s1) - (int)TO_LOWER(*s2);
  69. !     if (i != 0)
  70. !         return i;                /* this character different */
  71.       if (*s1 == NUL)
  72.           break;                /* strings match until NUL */
  73.       ++s1;
  74. ***************
  75. *** 977,994 ****
  76.   #if (!defined(HAVE_STRNCASECMP) && !defined(HAVE_STRNICMP)) || defined(PROTO)
  77.   /*
  78.    * Compare two strings, for length "len", ignoring case.
  79. !  * return 0 for match, 1 for difference
  80.    */
  81.       int
  82.   vim_strnicmp(s1, s2, len)
  83. !     char    *s1;
  84. !     char    *s2;
  85. !     size_t  len;
  86.   {
  87. !     while (len)
  88.       {
  89. !     if (TO_LOWER(*s1) != TO_LOWER(*s2))
  90. !         return 1;                /* this character different */
  91.       if (*s1 == NUL)
  92.           break;                /* strings match until NUL */
  93.       ++s1;
  94. --- 980,1000 ----
  95.   #if (!defined(HAVE_STRNCASECMP) && !defined(HAVE_STRNICMP)) || defined(PROTO)
  96.   /*
  97.    * Compare two strings, for length "len", ignoring case.
  98. !  * return 0 for match, < 0 for smaller, > 0 for bigger
  99.    */
  100.       int
  101.   vim_strnicmp(s1, s2, len)
  102. !     char    *s1;
  103. !     char    *s2;
  104. !     size_t    len;
  105.   {
  106. !     int        i;
  107. ! 
  108. !     while (len > 0)
  109.       {
  110. !     i = (int)TO_LOWER(*s1) - (int)TO_LOWER(*s2);
  111. !     if (i != 0)
  112. !         return i;                /* this character different */
  113.       if (*s1 == NUL)
  114.           break;                /* strings match until NUL */
  115.       ++s1;
  116. *** ../vim-5.4.35/src/version.h    Wed Aug 18 13:05:54 1999
  117. --- src/version.h    Thu Aug 19 11:51:10 1999
  118. ***************
  119. *** 19,26 ****
  120.   #define VIM_VERSION_MINOR_STR        "4"
  121.   #define VIM_VERSION_BUILD         57
  122.   #define VIM_VERSION_BUILD_STR        "57"
  123. ! #define VIM_VERSION_PATCHLEVEL         35
  124. ! #define VIM_VERSION_PATCHLEVEL_STR    "35"
  125.   
  126.   /*
  127.    * VIM_VERSION_NODOT is used for the runtime directory name.
  128. --- 19,26 ----
  129.   #define VIM_VERSION_MINOR_STR        "4"
  130.   #define VIM_VERSION_BUILD         57
  131.   #define VIM_VERSION_BUILD_STR        "57"
  132. ! #define VIM_VERSION_PATCHLEVEL         36
  133. ! #define VIM_VERSION_PATCHLEVEL_STR    "36"
  134.   
  135.   /*
  136.    * VIM_VERSION_NODOT is used for the runtime directory name.
  137. ***************
  138. *** 30,35 ****
  139.    */
  140.   #define VIM_VERSION_NODOT    "vim54"
  141.   #define VIM_VERSION_SHORT    "5.4"
  142. ! #define VIM_VERSION_MEDIUM    "5.4.35"
  143. ! #define VIM_VERSION_LONG    "VIM - Vi IMproved 5.4.35 (1999 Aug 18)"
  144. ! #define VIM_VERSION_LONG_DATE    "VIM - Vi IMproved 5.4.35 (1999 Aug 18, compiled "
  145. --- 30,35 ----
  146.    */
  147.   #define VIM_VERSION_NODOT    "vim54"
  148.   #define VIM_VERSION_SHORT    "5.4"
  149. ! #define VIM_VERSION_MEDIUM    "5.4.36"
  150. ! #define VIM_VERSION_LONG    "VIM - Vi IMproved 5.4.36 (1999 Aug 19)"
  151. ! #define VIM_VERSION_LONG_DATE    "VIM - Vi IMproved 5.4.36 (1999 Aug 19, compiled "
  152.  
  153. --
  154. hundred-and-one symptoms of being an internet addict:
  155. 147. You finally give up smoking...because it made the monitor dirty.
  156.  
  157. --/-/---- Bram Moolenaar ---- Bram@moolenaar.net ---- Bram@vim.org ---\-\--
  158.   \ \    www.vim.org/iccf      www.moolenaar.net       www.vim.org    / /
  159.