mvdXML 1.2 grammar mistake

buildingSMART / mvdXML

Repository used by the mvdXML group to support mvdXML specification and implementation.

28 stars 10 forks source link

Closed pjanck closed 3 years ago

pjanck commented 3 years ago

Currently:

boolean_expression
: ( NOT ) boolean_term (logical_interconnection ( NOT ) boolean_term)* ;

Missing ? for the (NOT) part - or is it a must?

Proposal:

boolean_expression
: ( NOT )? boolean_term (logical_interconnection ( NOT )? boolean_term)* ;

pjanck commented 3 years ago

I can prepare a PR after #19 and #21 have been merged (for easier traceability).

MatthiasWeise commented 3 years ago

Closed by #21