Skip to content

make buildInlinePattern support for '/'#234

Open
caogecym wants to merge 1 commit into
evilstreak:masterfrom
caogecym:master
Open

make buildInlinePattern support for '/'#234
caogecym wants to merge 1 commit into
evilstreak:masterfrom
caogecym:master

Conversation

@caogecym

Copy link
Copy Markdown

No description provided.

@ashb

ashb commented Jan 19, 2015

Copy link
Copy Markdown
Collaborator

I wasn't aware that '/' needs to be escaped. How to other parsers handle this?

If other parsers behave like this can you add a test case for this? Should be quite easy to expand this

test( "inline_escape", function(t, md) {
t.equivalent( md.processInline("\\bar"), [ "\\bar" ], "invalid escape" );
t.equivalent( md.processInline("\\>"), [ ">" ], "escapes >" );
t.equivalent( md.processInline("\\*foo*"), [ "*foo*" ], "escaped em" );
});

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

Successfully merging this pull request may close these issues.

2 participants