Re: [PATCH 03/17] qapi: Fix crash on stray double quote character

2023-05-09 Thread Markus Armbruster
Juan Quintela writes: > Markus Armbruster wrote: >> When the lexer chokes on a stray character, its shows the characters >> until the next structural character in the error message. > > You have a problem > >> It uses a regular expression > > You use regular expresions. > > Now you have two prob

Re: [PATCH 03/17] qapi: Fix crash on stray double quote character

2023-04-28 Thread Juan Quintela
Markus Armbruster wrote: > When the lexer chokes on a stray character, its shows the characters > until the next structural character in the error message. You have a problem > It uses a regular expression You use regular expresions. Now you have two problems. Yes, I had to do it. > to match

[PATCH 03/17] qapi: Fix crash on stray double quote character

2023-04-28 Thread Markus Armbruster
When the lexer chokes on a stray character, its shows the characters until the next structural character in the error message. It uses a regular expression to match a non-empty string of non-structural characters. Bug: the regular expression treats '"' as structural. When the lexer chokes on '"',