zorba-coders team mailing list archive
-
zorba-coders team
-
Mailing list archive
-
Message #19914
Re: [Merge] lp:~zorba-coders/zorba/bug-1130998 into lp:zorba
Review: Needs Fixing
Unfortunately this fix isn't in the right place. By changing the CharRef regexp, you've broken some valid XML character references like  . Also, the error message for � is not very good.
Nicolae - can you help point us in the right direction here? There is already code in xquery_scanner.l and jsoniq_scanner.l that verifies character references in string literals:
{StringLiteral} { if (checkXmlRefs(&yylval->err, yytext, yyleng, this, yylloc)) return token::UNRECOGNIZED; TRY_STRING_LITERAL(STRING_LITERAL, yytext, yyleng); }
However, same check also needs to be done anywhere else that may have character references, including element content and attribute values. But we don't know enough about flex/bison to understand how to get this done.
If possible I'd still like Luis to do the actual change, if only to help spread some knowledge about the lexer/parser to other team members. But if you could briefly describe what kind of change needs to happen, we'd appreciate it!
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1130998/+merge/156700
Your team Zorba Coders is subscribed to branch lp:zorba.
References