Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Double-decoding issue leading to difficulty to output <! in emu-grammar #46

Closed
claudepache opened this issue Nov 25, 2015 · 3 comments
Closed

Comments

@claudepache
Copy link

I don’t know if this is an issue of ecmarkdown proper or an interaction with some pre- or post-processing.

When working on the ES spec, I've noticed that the following code will not work as expected:

<emu-grammar>A :: `<!`</emu-grammar> 
<emu-grammar>A :: `&lt;!`</emu-grammar> 
<emu-grammar>A :: `&lt;&#x21;`</emu-grammar> 

It works when double-encoding one of the characters:

<emu-grammar>A :: `<&amp;#x21;`</emu-grammar> 
<emu-grammar>A :: `&amp;lt;!`</emu-grammar>

That makes very cumbersome to output the <! token in a grammar.

@bterlson
Copy link
Member

This is some interaction between emu and gmd. Not sure why I need to escape input to gmd, but I do. Fixed in latest Emu release, though!

@bterlson bterlson reopened this Dec 30, 2015
@bterlson
Copy link
Member

I was too hasty. The issue appears to be that <! is gmd syntax for lookahead doesn't contain and it gets confused. I'll file a bug over there (and revert the emu change).

@bterlson
Copy link
Member

Actually, two issues. For some reason HTML parsers see the following as a comment:

<emu-grammar>A :: `<!`</emu-grammar> 

Removing that case, the other issue is that grammarkdown wasn't emitting escaped html. I've already reported that bug and got it fixed, but didn't update the emu package.json to pick up the new version. I've made the appropriate updates and this is now ACTUALLY closed :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants