Обсуждение: pgsql: Added just another test case.

Поиск
Список
Период
Сортировка

pgsql: Added just another test case.

От
meskes@postgresql.org (Michael Meskes)
Дата:
Log Message:
-----------
 Added just another test case.
 Fixed missing continuation line character.
 Do not translate $-quoting.
 Bit field notation belongs to a variable not a variable list.
 Output of line number only done by one function.

Modified Files:
--------------
    pgsql/src/interfaces/ecpg:
        ChangeLog (r1.299 -> r1.300)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/ChangeLog.diff?r1=1.299&r2=1.300)
    pgsql/src/interfaces/ecpg/preproc:
        ecpg.c (r1.93 -> r1.94)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/preproc/ecpg.c.diff?r1=1.93&r2=1.94)
        output.c (r1.13 -> r1.14)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/preproc/output.c.diff?r1=1.13&r2=1.14)
        pgc.l (r1.142 -> r1.143)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/preproc/pgc.l.diff?r1=1.142&r2=1.143)
        preproc.y (r1.319 -> r1.320)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/preproc/preproc.y.diff?r1=1.319&r2=1.320)
    pgsql/src/interfaces/ecpg/test:
        Makefile (r1.52 -> r1.53)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/test/Makefile.diff?r1=1.52&r2=1.53)
        perftest.pgc (r1.9 -> r1.10)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/test/perftest.pgc.diff?r1=1.9&r2=1.10)
        test1.pgc (r1.33 -> r1.34)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/test/test1.pgc.diff?r1=1.33&r2=1.34)
        test5.pgc (r1.7 -> r1.8)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/test/test5.pgc.diff?r1=1.7&r2=1.8)

Added Files:
-----------
    pgsql/src/interfaces/ecpg/test:
        test_func.pgc (r1.1)

(http://developer.postgresql.org/cvsweb.cgi/pgsql/src/interfaces/ecpg/test/test_func.pgc?rev=1.1&content-type=text/x-cvsweb-markup)

Re: pgsql: Added just another test case.

От
Tom Lane
Дата:
meskes@postgresql.org (Michael Meskes) writes:
> Log Message:
> -----------
>  Added just another test case.
>  Fixed missing continuation line character.
>  Do not translate $-quoting.
>  Bit field notation belongs to a variable not a variable list.
>  Output of line number only done by one function.

This commit seems to have broken HEAD:

bison -y -d  preproc.y
mv -f y.tab.c ./preproc.c
mv -f y.tab.h ./preproc.h
/usr/bin/flex  -o'pgc.c' pgc.l
ccache gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline -Wdeclaration-after-statement -Wendif-labels
-fno-strict-aliasing-g -Wno-error -pthread  -D_REENTRANT -D_THREAD_SAFE -D_POSIX_PTHREAD_SEMANTICS -I./../include -I.
-I../../../../src/include-D_GNU_SOURCE  -I/usr/include/et -DMAJOR_VERSION=4 -DMINOR_VERSION=2 -DPATCHLEVEL=1  -c -o
preproc.opreproc.c 
In file included from preproc.y:6614:
pgc.c: In function `yylex':
pgc.c:1522: warning: label `find_rule' defined but not used
preproc.y: At top level:
pgc.c:3678: warning: 'yy_flex_realloc' defined but not used
ccache gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline -Wdeclaration-after-statement -Wendif-labels
-fno-strict-aliasing-g -Wno-error -pthread  -D_REENTRANT -D_THREAD_SAFE -D_POSIX_PTHREAD_SEMANTICS -I./../include -I.
-I../../../../src/include-D_GNU_SOURCE  -I/usr/include/et -DMAJOR_VERSION=4 -DMINOR_VERSION=2 -DPATCHLEVEL=1  -c -o
type.otype.c 
ccache gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline -Wdeclaration-after-statement -Wendif-labels
-fno-strict-aliasing-g -Wno-error -pthread  -D_REENTRANT -D_THREAD_SAFE -D_POSIX_PTHREAD_SEMANTICS -I./../include -I.
-I../../../../src/include-D_GNU_SOURCE  -I/usr/include/et -DMAJOR_VERSION=4 -DMINOR_VERSION=2 -DPATCHLEVEL=1  -c -o
ecpg.oecpg.c 
ccache gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline -Wdeclaration-after-statement -Wendif-labels
-fno-strict-aliasing-g -Wno-error -pthread  -D_REENTRANT -D_THREAD_SAFE -D_POSIX_PTHREAD_SEMANTICS -I./../include -I.
-I../../../../src/include-D_GNU_SOURCE  -I/usr/include/et -DMAJOR_VERSION=4 -DMINOR_VERSION=2 -DPATCHLEVEL=1  -c -o
ecpg_keywords.oecpg_keywords.c 
ccache gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline -Wdeclaration-after-statement -Wendif-labels
-fno-strict-aliasing-g -Wno-error -pthread  -D_REENTRANT -D_THREAD_SAFE -D_POSIX_PTHREAD_SEMANTICS -I./../include -I.
-I../../../../src/include-D_GNU_SOURCE  -I/usr/include/et -DMAJOR_VERSION=4 -DMINOR_VERSION=2 -DPATCHLEVEL=1  -c -o
output.ooutput.c 
output.c: In function `hashline_number':
output.c:90: error: `yydebug' undeclared (first use in this function)
output.c:90: error: (Each undeclared identifier is reported only once
output.c:90: error: for each function it appears in.)
make[4]: *** [output.o] Error 1

            regards, tom lane

Re: pgsql: Added just another test case.

От
Bruce Momjian
Дата:
Yes, it is the yydebug check.  I am thinking we should just comment out
the check until we get it fixed.

---------------------------------------------------------------------------

Tom Lane wrote:
> meskes@postgresql.org (Michael Meskes) writes:
> > Log Message:
> > -----------
> >  Added just another test case.
> >  Fixed missing continuation line character.
> >  Do not translate $-quoting.
> >  Bit field notation belongs to a variable not a variable list.
> >  Output of line number only done by one function.
>
> This commit seems to have broken HEAD:
>
> bison -y -d  preproc.y
> mv -f y.tab.c ./preproc.c
> mv -f y.tab.h ./preproc.h
> /usr/bin/flex  -o'pgc.c' pgc.l
> ccache gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline -Wdeclaration-after-statement -Wendif-labels
-fno-strict-aliasing-g -Wno-error -pthread  -D_REENTRANT -D_THREAD_SAFE -D_POSIX_PTHREAD_SEMANTICS -I./../include -I.
-I../../../../src/include-D_GNU_SOURCE  -I/usr/include/et -DMAJOR_VERSION=4 -DMINOR_VERSION=2 -DPATCHLEVEL=1  -c -o
preproc.opreproc.c 
> In file included from preproc.y:6614:
> pgc.c: In function `yylex':
> pgc.c:1522: warning: label `find_rule' defined but not used
> preproc.y: At top level:
> pgc.c:3678: warning: 'yy_flex_realloc' defined but not used
> ccache gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline -Wdeclaration-after-statement -Wendif-labels
-fno-strict-aliasing-g -Wno-error -pthread  -D_REENTRANT -D_THREAD_SAFE -D_POSIX_PTHREAD_SEMANTICS -I./../include -I.
-I../../../../src/include-D_GNU_SOURCE  -I/usr/include/et -DMAJOR_VERSION=4 -DMINOR_VERSION=2 -DPATCHLEVEL=1  -c -o
type.otype.c 
> ccache gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline -Wdeclaration-after-statement -Wendif-labels
-fno-strict-aliasing-g -Wno-error -pthread  -D_REENTRANT -D_THREAD_SAFE -D_POSIX_PTHREAD_SEMANTICS -I./../include -I.
-I../../../../src/include-D_GNU_SOURCE  -I/usr/include/et -DMAJOR_VERSION=4 -DMINOR_VERSION=2 -DPATCHLEVEL=1  -c -o
ecpg.oecpg.c 
> ccache gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline -Wdeclaration-after-statement -Wendif-labels
-fno-strict-aliasing-g -Wno-error -pthread  -D_REENTRANT -D_THREAD_SAFE -D_POSIX_PTHREAD_SEMANTICS -I./../include -I.
-I../../../../src/include-D_GNU_SOURCE  -I/usr/include/et -DMAJOR_VERSION=4 -DMINOR_VERSION=2 -DPATCHLEVEL=1  -c -o
ecpg_keywords.oecpg_keywords.c 
> ccache gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Winline -Wdeclaration-after-statement -Wendif-labels
-fno-strict-aliasing-g -Wno-error -pthread  -D_REENTRANT -D_THREAD_SAFE -D_POSIX_PTHREAD_SEMANTICS -I./../include -I.
-I../../../../src/include-D_GNU_SOURCE  -I/usr/include/et -DMAJOR_VERSION=4 -DMINOR_VERSION=2 -DPATCHLEVEL=1  -c -o
output.ooutput.c 
> output.c: In function `hashline_number':
> output.c:90: error: `yydebug' undeclared (first use in this function)
> output.c:90: error: (Each undeclared identifier is reported only once
> output.c:90: error: for each function it appears in.)
> make[4]: *** [output.o] Error 1
>
>             regards, tom lane
>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: don't forget to increase your free space map settings
>

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073