search

tjhancocks / kestrel-development-kit

A Development Kit for the Kestrel Game Engine
MIT License
13 stars 1 forks source link

Setup diagnostic reporting for errors/warnings in kas. #1

Closed tjhancocks closed 4 years ago

tjhancocks commented 4 years ago

Is your feature request related to a problem? Please describe. Currently the way kas is reporting errors and warnings to the users as it performs lexical analysis, semantic analysis, assembly etc is not particularly useful. It doesn't always reflect what has happened, and doesn't fail gracefully.

Describe the solution you'd like Diagnostic messages should indicate the severity of the message, the file it was raised in, and the line number. For example:

ships.kdl : L10 : Warning - Resource instances should have an ID specified.

Describe alternatives you've considered None considered - open to suggestions.

Additional context N/a

tjhancocks commented 4 years ago

This is mostly completed, with the exception of the assemblers not having it yet. However due to the possible reworking of the assemblers, I'm closing this for now, as the default should be to use the new logging mechanism for new code.