home *** CD-ROM | disk | FTP | other *** search
/ OS/2 Shareware BBS: 22 gnu / 22-gnu.zip / GNUGRE.ZIP / grep16.bug < prev    next >
Internet Message Format  |  1992-07-18  |  4KB
  1. From NewsServ!reseq!sol.ctr.columbia.edu!zaphod.mps.ohio-state.edu!cis.ohio-state.edu!twinsun.COM!eggert Wed Jun 10 02:22:56 MESZ 1992
  2. Article: 3020 of gnu.utils.bug
  3. Path: NewsServ!reseq!sol.ctr.columbia.edu!zaphod.mps.ohio-state.edu!cis.ohio-state.edu!twinsun.COM!eggert
  4. From: eggert@twinsun.COM (Paul Eggert)
  5. Newsgroups: gnu.utils.bug
  6. Subject: grep 1.6 issues wrong error messages for some bad patterns
  7. Message-ID: <9206090003.AA29600@farside.twinsun.com>
  8. Date: 9 Jun 92 00:03:51 GMT
  9. Sender: gnulists@ai.mit.edu
  10. Distribution: gnu
  11. Organization: GNUs Not Usenet
  12. Lines: 37
  13. Approved: bug-gnu-utils@prep.ai.mit.edu
  14.  
  15. With the -w and -x options, grep 1.6 issues the wrong error messages
  16. for some bad patterns.  For example:
  17.  
  18.     $ grep -x '\'
  19.     grep: Unbalanced (
  20.     $ egrep -x ')('
  21.     egrep: Unmatched \)
  22.  
  23. The problem is that the regular expression syntax is not being checked
  24. at the proper point.  Here is a patch.
  25.  
  26. *** old/grep.c    Tue May  5 19:40:20 1992
  27. --- new/grep.c    Mon Jun  8 16:57:49 1992
  28. ***************
  29. *** 724,729 ****
  30. --- 724,732 ----
  31.       }
  32.     else
  33.       regexp_len = strlen(the_regexp);
  34. + 
  35. +   if (regex_errmesg = re_compile_pattern(the_regexp, regexp_len, ®ex))
  36. +     regerror(regex_errmesg);
  37.     
  38.     if (whole_word || whole_line)
  39.       {
  40. ***************
  41. *** 768,776 ****
  42.     else
  43.       regcompile(the_regexp, regexp_len, ®, 1);
  44.   
  45. -   
  46. -   if (regex_errmesg = re_compile_pattern(the_regexp, regexp_len, ®ex))
  47. -     regerror(regex_errmesg);
  48.     
  49.     /*
  50.       Find the longest metacharacter-free string which must occur in the
  51. --- 771,776 ----
  52.  
  53.  
  54. From NewsServ!reseq!sol.ctr.columbia.edu!zaphod.mps.ohio-state.edu!cis.ohio-state.edu!twinsun.COM!eggert Wed Jun 10 02:22:27 MESZ 1992
  55. Article: 3018 of gnu.utils.bug
  56. Path: NewsServ!reseq!sol.ctr.columbia.edu!zaphod.mps.ohio-state.edu!cis.ohio-state.edu!twinsun.COM!eggert
  57. From: eggert@twinsun.COM (Paul Eggert)
  58. Newsgroups: gnu.utils.bug
  59. Subject: grep 1.6 -w silently overrides -x
  60. Message-ID: <9206090014.AA29620@farside.twinsun.com>
  61. Date: 9 Jun 92 00:14:36 GMT
  62. Sender: gnulists@ai.mit.edu
  63. Distribution: gnu
  64. Organization: GNUs Not Usenet
  65. Lines: 86
  66. Approved: bug-gnu-utils@prep.ai.mit.edu
  67.  
  68. In GNU grep 1.6, the -w option silently overrides the -x option.
  69. But from the documentation, one would expect -w -x to cause the pattern
  70. to match only if the match is a whole word _and_ a whole line.
  71. For example, if the input is
  72.  
  73. a
  74. a b c
  75.  
  76. then `grep -w -x a' should output just the first line `a';
  77. but currently it outputs both lines.
  78.  
  79. Here is an (untested) patch.
  80.  
  81. *** old/grep.c    Tue May  5 19:40:20 1992
  82. --- new/grep.c    Mon Jun  8 17:12:26 1992
  83. ***************
  84. *** 734,766 ****
  85.        BUG: Using [A-Za-z_] is locale-dependent!  */
  86.   
  87.         char *n = malloc(regexp_len + 50);
  88. !       int i = 0;
  89.   
  90.   #ifdef EGREP
  91.         if (whole_word)
  92. !     strcpy(n, "(^|[^A-Za-z_])(");
  93. !       else
  94. !     strcpy(n, "^(");
  95.   #else
  96.         /* Todo:  Make *sure* this is the right syntax.  Down with grep! */
  97.         if (whole_word)
  98. !     strcpy(n, "\\(^\\|[^A-Za-z_]\\)\\(");
  99. !       else
  100. !     strcpy(n, "^\\(");
  101.   #endif
  102.         i = strlen(n);
  103.         bcopy(the_regexp, n + i, regexp_len);
  104.         i += regexp_len;
  105.   #ifdef EGREP
  106.         if (whole_word)
  107. !     strcpy(n + i, ")([^A-Za-z_]|$)");
  108. !       else
  109. !     strcpy(n + i, ")$");
  110.   #else
  111.         if (whole_word)
  112. !     strcpy(n + i, "\\)\\([^A-Za-z_]\\|$\\)");
  113. !       else
  114. !     strcpy(n + i, "\\)$");
  115.   #endif
  116.         i += strlen(n + i);
  117.         regcompile(n, i, ®, 1);
  118. --- 737,771 ----
  119.        BUG: Using [A-Za-z_] is locale-dependent!  */
  120.   
  121.         char *n = malloc(regexp_len + 50);
  122. !       int i;
  123.   
  124. +       n[0] = 0;
  125.   #ifdef EGREP
  126. +       if (whole_line)
  127. +     strcat(n, "^(");
  128.         if (whole_word)
  129. !     strcat(n, "(^|[^A-Za-z_])(");
  130.   #else
  131.         /* Todo:  Make *sure* this is the right syntax.  Down with grep! */
  132. +       if (whole_line)
  133. +     strcat(n, "^\\(");
  134.         if (whole_word)
  135. !     strcat(n, "\\(^\\|[^A-Za-z_]\\)\\(");
  136.   #endif
  137.         i = strlen(n);
  138.         bcopy(the_regexp, n + i, regexp_len);
  139.         i += regexp_len;
  140. +       n[i] = 0;
  141.   #ifdef EGREP
  142.         if (whole_word)
  143. !     strcat(n + i, ")([^A-Za-z_]|$)");
  144. !       if (whole_line)
  145. !     strcat(n + i, ")$");
  146.   #else
  147.         if (whole_word)
  148. !     strcat(n + i, "\\)\\([^A-Za-z_]\\|$\\)");
  149. !       if (whole_line)
  150. !     strcat(n + i, "\\)$");
  151.   #endif
  152.         i += strlen(n + i);
  153.         regcompile(n, i, ®, 1);
  154.  
  155.  
  156.