search

lonely-lockley / archinsight

Simplicity in Code, Power in Design
https://archinsight.org
GNU Affero General Public License v3.0
51 stars 5 forks source link

Change Insight parser - move required parameters check to Translator #84

Closed lonely-lockley closed 4 months ago

lonely-lockley commented 4 months ago

This rule in parser

namedEntityParametersDeclaration
    :    ( nameParameter descriptionParameter? technologyParameter?
         | descriptionParameter? nameParameter technologyParameter?
         | descriptionParameter? technologyParameter? nameParameter
         | nameParameter technologyParameter? descriptionParameter?
         | technologyParameter? nameParameter descriptionParameter?
         | technologyParameter? descriptionParameter? nameParameter
         )
    ;

makes it very hard to add new parameters and generates misleading error messages. Parser or Linker in translator service would do this job much better.