Draft: Remove 'syntax error, ' and internalise ParseError
Both the Vala and Genie parsers can throw a ParseError
. At present this is caught and has the phrase 'syntax error, ' placed in front of the exception message. This makes it more difficult for translations because the message is now split. It also seems superfluous. For example missing the semi-colon at the end of a line gives error: syntax error, expected `;'
. Well we know it is an error and the word 'syntax' doesn't add much. So this patch will make the message just error: expected `;'
If that is too brief then the individual message should be updated, but that is part of a wider review.
Some code that causes syntax error,
to be reported twice to the programmer has also been removed.
ParseError
has a FAILED
value. This is unused and so removed in the second patch. Given this changes the public API ParseError
has also been made internal. The second patch is a bit more controversial, but would help understand how third parties use libvala for checking source files and reporting diagnostics.