[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Bison 3.0.3 (stable)
From: |
Akim Demaille |
Subject: |
Re: Bison 3.0.3 (stable) |
Date: |
Sun, 18 Jan 2015 15:20:20 +0100 |
> Le 17 janv. 2015 à 21:21, John Horigan <address@hidden> a écrit :
>
> Hi Akim,
Hi John
Thanks for the report! Let's keep this public: cc'd to bug-bison.
> I tried bison 3.0.3 on my OSX Yosemite system with XCode 6.1.1
>
> I ran ./configure && make && make check
>
> Test 203 failed because it declared a loop variable inside a for loop.
I don't see that. Can you pinpoint the problem?
> So I ran make distclean && ./configure CC=c99 && make && make check
>
> During make there were four warnings:
> src/symtab.h:76:14: warning: ISO C forbids forward references to 'enum' types
> [-Wpedantic]
ok.
> src/graphviz.c:116:35: warning: positional arguments are not supported by ISO
> C [-Wformat-non-iso]
ok.
> lib/printf-frexp.c:72:3: warning: use of GNU statement expression extension
> [-Wgnu-statement-expression]
This is from gnulib, I'm sure it's used only when it's safe to use.
> lib/abitset.c:789:24: warning: using extended field designator is an
> extension [-Wextended-offsetof]
It's been ok for years, so I'll ignore that warning.
> Test 203 still fails. But now I don't understand why. I have attached
> testsuite.log
The relevant bit is:
./headers.at:262: $CXX $CXXFLAGS $CPPFLAGS -c -o cxx-only.o cxx-only.cc
stderr:
g++: warning: couldn't understand kern.osversion '14.0.0
stdout:
./headers.at:262: $CXX $CXXFLAGS $CPPFLAGS $LDFLAGS c-only.o cxx-only.o -o
c-and-cxx ||
exit 77
--- /dev/null 2015-01-17 11:45:15.000000000 -0800
+++ /Users/john/Projects/bison-3.0.3/tests/testsuite.dir/at-groups/203/stderr
2015-01-17 11:45:18.000000000 -0800
@@ -0,0 +1 @@
+g++: warning: couldn't understand kern.osversion '14.0.0
stdout:
203. headers.at:112: 203. Several parsers (headers.at:112): FAILED
(headers.at:262)
Maybe you need to reinstall g++ after you upgraded to Yosemite?
http://stackoverflow.com/questions/26486163/gcc-4-9-1-in-os-x-yosemite-gcc-warning-couldn-t-understand-kern-osversion-1
I'll install this:
commit 6ee80d5417e1a7493d69f55305da0941bb40e105
Author: Akim Demaille <address@hidden>
Date: Sun Jan 18 15:18:38 2015 +0100
build: fix some warnings
Reported by John Horigan.
* src/graphviz.c, src/symtab.h: Address compiler warnings.
diff --git a/THANKS b/THANKS
index 875ac96..26bea0a 100644
--- a/THANKS
+++ b/THANKS
@@ -66,6 +66,7 @@ Jim Kent address@hidden
Jim Meyering address@hidden
Joel E. Denny address@hidden
Johan van Selst address@hidden
+John Horigan address@hidden
Jonathan Fabrizio address@hidden
Jonathan Nieder address@hidden
Juan Manuel Guerrero address@hidden
diff --git a/src/graphviz.c b/src/graphviz.c
index e7611b0..663cf37 100644
--- a/src/graphviz.c
+++ b/src/graphviz.c
@@ -113,8 +113,8 @@ conclude_red (struct obstack *out, int source, rule_number
ruleno,
with n the source state and m the rule number. This is because we
don't want all the reductions bearing a same rule number to point to
the same state, since that is not the desired format. */
- fprintf (fout, " %1$d -> \"%1$dR%2$d%3$s\" [",
- source, ruleno, ed);
+ fprintf (fout, " %d -> \"%dR%d%s\" [",
+ source, source, ruleno, ed);
/* (The lookahead tokens have been added to the beginning of the
obstack, in the caller function.) */
diff --git a/src/symtab.h b/src/symtab.h
index aa6d65b..f92407f 100644
--- a/src/symtab.h
+++ b/src/symtab.h
@@ -73,12 +73,12 @@ typedef enum
declared,
} status;
-typedef enum code_props_type code_props_type;
enum code_props_type
{
destructor = 0,
printer = 1,
};
+typedef enum code_props_type code_props_type;
enum { CODE_PROPS_SIZE = 2 };
- Bison 3.0.3 (stable), Akim Demaille, 2015/01/15
- Message not available
- Re: Bison 3.0.3 (stable),
Akim Demaille <=
- Re: Bison 3.0.3 (stable), Hans Aberg, 2015/01/18
- Re: Bison 3.0.3 (stable), Akim Demaille, 2015/01/18
- Re: Bison 3.0.3 (stable), Hans Aberg, 2015/01/18
- Re: Bison 3.0.3 (stable), John Horigan, 2015/01/18
- Re: Bison 3.0.3 (stable), Akim Demaille, 2015/01/19
Re: Bison 3.0.3 (stable), Thomas Jahns, 2015/01/19