Re: svn commit: r214964 - head/usr.bin/yacc

2010-11-08 Thread Alexander Best 
On Sun Nov  7 10, David E. O'Brien wrote:
 Author: obrien
 Date: Sun Nov  7 23:44:40 2010
 New Revision: 214964
 URL: http://svn.freebsd.org/changeset/base/214964
 
 Log:
   Add '-y' for bison compatibility.
   
   Obtained from:  http://invisible-island.net

could you take a look at PR 140309? it's a simple one line fix to prevent yacc
from segfaulting in certain cases.

cheers.
alex

 
 Modified:
   head/usr.bin/yacc/main.c
   head/usr.bin/yacc/yacc.1
 
 Modified: head/usr.bin/yacc/main.c
 ==
 --- head/usr.bin/yacc/main.c  Sun Nov  7 23:34:05 2010(r214963)
 +++ head/usr.bin/yacc/main.c  Sun Nov  7 23:44:40 2010(r214964)
 @@ -166,7 +166,7 @@ getargs(int argc, char *argv[])
  {
  int ch;
  
 -while ((ch = getopt(argc, argv, b:dlo:p:rtv)) != -1)
 +while ((ch = getopt(argc, argv, b:dlo:p:rtvy)) != -1)
  {
   switch (ch)
   {
 @@ -202,6 +202,10 @@ getargs(int argc, char *argv[])
   vflag = 1;
   break;
  
 + case 'y':
 + /* for bison compatibility -- byacc is already POSIX compatible */
 + break;
 +
   default:
   usage();
   }
 
 Modified: head/usr.bin/yacc/yacc.1
 ==
 --- head/usr.bin/yacc/yacc.1  Sun Nov  7 23:34:05 2010(r214963)
 +++ head/usr.bin/yacc/yacc.1  Sun Nov  7 23:44:40 2010(r214964)
 @@ -44,7 +44,7 @@
  .Nd an LALR(1) parser generator
  .Sh SYNOPSIS
  .Nm
 -.Op Fl dlrtv
 +.Op Fl dlrtvy
  .Op Fl b Ar file_prefix
  .Op Fl o Ar output_filename
  .Op Fl p Ar symbol_prefix
 @@ -118,6 +118,10 @@ so that debugging statements will be inc
  Cause a human-readable description of the generated parser to
  be written to the file
  .Pa y.output .
 +.It Fl y
 +NOOP for bison compatibility.
 +.Nm
 +is already designed to be POSIX yacc compatible.
  .El
  .Sh ENVIRONMENT
  .Bl -tag -width .Ev TMPDIR

-- 
a13x
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org

Re: svn commit: r214964 - head/usr.bin/yacc

2010-11-08 Thread Alexander Best 
On Tue Nov  9 10, Alexander Best wrote:
 On Sun Nov  7 10, David E. O'Brien wrote:
  Author: obrien
  Date: Sun Nov  7 23:44:40 2010
  New Revision: 214964
  URL: http://svn.freebsd.org/changeset/base/214964
  
  Log:
Add '-y' for bison compatibility.

Obtained from:http://invisible-island.net
 
 could you take a look at PR 140309? it's a simple one line fix to prevent yacc
 from segfaulting in certain cases.

fixing PR 23254 also doesn't seem too complicated.

cheers.
alex

 
 cheers.
 alex
 
  
  Modified:
head/usr.bin/yacc/main.c
head/usr.bin/yacc/yacc.1
  
  Modified: head/usr.bin/yacc/main.c
  ==
  --- head/usr.bin/yacc/main.cSun Nov  7 23:34:05 2010
  (r214963)
  +++ head/usr.bin/yacc/main.cSun Nov  7 23:44:40 2010
  (r214964)
  @@ -166,7 +166,7 @@ getargs(int argc, char *argv[])
   {
   int ch;
   
  -while ((ch = getopt(argc, argv, b:dlo:p:rtv)) != -1)
  +while ((ch = getopt(argc, argv, b:dlo:p:rtvy)) != -1)
   {
  switch (ch)
  {
  @@ -202,6 +202,10 @@ getargs(int argc, char *argv[])
  vflag = 1;
  break;
   
  +   case 'y':
  +   /* for bison compatibility -- byacc is already POSIX compatible */
  +   break;
  +
  default:
  usage();
  }
  
  Modified: head/usr.bin/yacc/yacc.1
  ==
  --- head/usr.bin/yacc/yacc.1Sun Nov  7 23:34:05 2010
  (r214963)
  +++ head/usr.bin/yacc/yacc.1Sun Nov  7 23:44:40 2010
  (r214964)
  @@ -44,7 +44,7 @@
   .Nd an LALR(1) parser generator
   .Sh SYNOPSIS
   .Nm
  -.Op Fl dlrtv
  +.Op Fl dlrtvy
   .Op Fl b Ar file_prefix
   .Op Fl o Ar output_filename
   .Op Fl p Ar symbol_prefix
  @@ -118,6 +118,10 @@ so that debugging statements will be inc
   Cause a human-readable description of the generated parser to
   be written to the file
   .Pa y.output .
  +.It Fl y
  +NOOP for bison compatibility.
  +.Nm
  +is already designed to be POSIX yacc compatible.
   .El
   .Sh ENVIRONMENT
   .Bl -tag -width .Ev TMPDIR
 
 -- 
 a13x

-- 
a13x
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org

svn commit: r214964 - head/usr.bin/yacc

2010-11-07 Thread David E. O'Brien 
Author: obrien
Date: Sun Nov  7 23:44:40 2010
New Revision: 214964
URL: http://svn.freebsd.org/changeset/base/214964

Log:
  Add '-y' for bison compatibility.
  
  Obtained from:http://invisible-island.net

Modified:
  head/usr.bin/yacc/main.c
  head/usr.bin/yacc/yacc.1

Modified: head/usr.bin/yacc/main.c
==
--- head/usr.bin/yacc/main.cSun Nov  7 23:34:05 2010(r214963)
+++ head/usr.bin/yacc/main.cSun Nov  7 23:44:40 2010(r214964)
@@ -166,7 +166,7 @@ getargs(int argc, char *argv[])
 {
 int ch;
 
-while ((ch = getopt(argc, argv, b:dlo:p:rtv)) != -1)
+while ((ch = getopt(argc, argv, b:dlo:p:rtvy)) != -1)
 {
switch (ch)
{
@@ -202,6 +202,10 @@ getargs(int argc, char *argv[])
vflag = 1;
break;
 
+   case 'y':
+   /* for bison compatibility -- byacc is already POSIX compatible */
+   break;
+
default:
usage();
}

Modified: head/usr.bin/yacc/yacc.1
==
--- head/usr.bin/yacc/yacc.1Sun Nov  7 23:34:05 2010(r214963)
+++ head/usr.bin/yacc/yacc.1Sun Nov  7 23:44:40 2010(r214964)
@@ -44,7 +44,7 @@
 .Nd an LALR(1) parser generator
 .Sh SYNOPSIS
 .Nm
-.Op Fl dlrtv
+.Op Fl dlrtvy
 .Op Fl b Ar file_prefix
 .Op Fl o Ar output_filename
 .Op Fl p Ar symbol_prefix
@@ -118,6 +118,10 @@ so that debugging statements will be inc
 Cause a human-readable description of the generated parser to
 be written to the file
 .Pa y.output .
+.It Fl y
+NOOP for bison compatibility.
+.Nm
+is already designed to be POSIX yacc compatible.
 .El
 .Sh ENVIRONMENT
 .Bl -tag -width .Ev TMPDIR
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org