Build bison warning

Lionel Elie Mamane lionel at mamane.lu
Tue Sep 23 05:23:54 PDT 2014


On Mon, Sep 22, 2014 at 08:38:48AM -0500, Norbert Thiebaud wrote:
> On Mon, Sep 22, 2014 at 3:07 AM, Stephan Bergmann <sbergman at redhat.com> wrote:
>> On 09/21/2014 08:29 AM, julien2412 wrote:

>>> /home/julien/compile-libreoffice/libreoffice/connectivity/source/parse/sqlbison.y:
>>> warning: 13 shift/reduce conflicts [-Wconflicts-sr]

>>> /home/julien/compile-libreoffice/libreoffice/connectivity/source/parse/sqlbison.y:
>>> warning: 15 reduce/reduce conflicts [-Wconflicts-rr]

>> [...]

>>> What about reduce and shift conflicts, any bison expert here?

>> Shift/reduce and (for GLR parsers, which that sqlbison.y is) reduce/reduce
>> conflicts are somewhat common and expected, so there's even %expect and
>> %expect-rr declarations that specify the number of expected sift/reduce and
>> reduce/reduce conflicts, respectively, and Bison will then only warn if the
>> numbers are wrong.  Though, of course, somebody familiar with that
>> sqlbison.y should probably have a look before blindly adding such
>> declarations to silence the warnings.

> yeah.. I advise _not_ hiding these. at this point it is highly
> unlikely that these be all 'legitimate'

Indeed; our SQL parser is not in the best of states.

-- 
Lionel


More information about the LibreOffice mailing list