csv-validator
csv-validator copied to clipboard
validator get stuck with malformed comment
This schema causes the validator (gui or cmd) to get stuck:
version 1.1
/* === SOME TEXT HERE that is quite long ===
*
*/
@noHeader @permitEmpty
The doc indeed says that asterisk cannot be used inside a multi-line comment. But the validator should nicely complain instead of getting stuck ;)
Looks like we have some ambiguity in the Grammar :-/
@adamretter as you commented on #61 an asterisk within a multiline comment is not allowed by the EBNF. Thomas can you confirm which version you're using as this issue seemed to have been fixed by something else last time I tried it.
Got bit by that as well with 1.1.5. In my case I was attempting to use a bit of markdown formatting.
To retest in current versions
I'm seeing this problem with the current version.
[2.1] failure: Invalid column definition
/* === SOME TEXT HERE that is quite long ===
^,None,None)
@DavidUnderdown - should we accept asterisks in the comments? Or would you prefer a different error message?
As commented above the grammar definition does prohibit asterisks within a multiline comment, so happy for an error to be thrown. It's not really an invalid column definition though, so it would be good to explicitly reference that the additional asterisk in the comment is the problem if possible.