Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Text-Markdown CPAN distribution.

Report information
The Basics
Id: 32951
Status: resolved
Priority: 0/
Queue: Text-Markdown

People
Owner: bobtfish [...] bobtfish.net
Requestors: petite.abeille [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Fwd: Entity conversion glitch?
Date: Tue, 5 Feb 2008 19:41:18 +0000
To: bug-Text-Markdown [...] rt.cpan.org
From: Tomas Doran <bobtfish [...] bobtfish.net>
Download (untitled) / with headers
text/plain 820b
Begin forwarded message: Show quoted text
> From: Petite Abeille <petite.abeille@gmail.com> > Date: 3 February 2008 17:22:31 GMT > To: markdown-discuss@six.pairlist.net > Subject: Entity conversion glitch? > Reply-To: "Discussion related to Markdown." <markdown- > discuss@six.pairlist.net> > > Hello, > > Given the following text: > > under a license from AT&T; however, others were based on BSD instead. > > Daring Fireball's Markdown Dingus produces: > > <p>under a license from AT&T; however, others were based on BSD > instead.</p> > > Note how the '&' is not escaped to '&amp;'. > > Bug? Feature? > > Thanks in advance. > > Kind regards, > > PA. > _______________________________________________ > Markdown-Discuss mailing list > Markdown-Discuss@six.pairlist.net > http://six.pairlist.net/mailman/listinfo/markdown-discuss
Download (untitled) / with headers
text/plain 578b
waylan@gmail.com said: I would say this is a minor bug, and an interesting edge case as well. The cause is the semi-colon following the `T`. `&T;` looks like an html entity. If you remove the semicolon, the `&` is properly converted to `&amp;`. In python-markdown, you can escape the `&` and you will get the expected output: under a license from AT\&T; however, others were based on BSD instead. becomes: <p>under a license from AT&amp;T; however, others were based on BSD instead. </p> Unfortunately, this does not seem to work in the other implementations.
Download (untitled) / with headers
text/plain 335b
Hiya I've added this as a bug to Text::Markdown on rt.cpan.org from your Markdown mailing list post. Are you using original Markdown.pl, one of the CPAN versions, or a non-perl version? Are you using the perl version (any of them), and so is this relevant to you - or would you like me to remove you from the ticket requestor field?
Subject: Re: [rt.cpan.org #32951] Fwd: Entity conversion glitch?
Date: Tue, 5 Feb 2008 21:15:39 +0100
To: bug-Text-Markdown [...] rt.cpan.org
From: Petite Abeille <petite.abeille [...] gmail.com>
Download (untitled) / with headers
text/plain 752b
Hi Tomas, On Feb 5, 2008, at 9:10 PM, Tomas Doran via RT wrote: Show quoted text
> <URL: http://rt.cpan.org/Ticket/Display.html?id=32951 > > > I've added this as a bug to Text::Markdown on rt.cpan.org from your > Markdown mailing list post. > > Are you using original Markdown.pl, one of the CPAN versions, or a > non-perl version? Are you using the perl version (any of them),
Nope. Using Niklas Frykholm's Lua implementation: http://www.frykholm.se/files/markdown.lua Show quoted text
> and so is this relevant to you - or would you like me to remove you > from the > ticket requestor field?
Considering that the perl implementation is the 'canonical' one, people tend to mimic its, hmmm, idiosyncrasies :) You can keep me posted. Thanks for filing a bug :) Cheers, PA.
Subject: Re: [rt.cpan.org #32951] Fwd: Entity conversion glitch?
Date: Tue, 5 Feb 2008 20:26:49 +0000
To: bug-Text-Markdown [...] rt.cpan.org
From: Tomas Doran <bobtfish [...] bobtfish.net>
Download (untitled) / with headers
text/plain 1.4k
On 5 Feb 2008, at 20:16, petite.abeille@gmail.com via RT wrote: Show quoted text
>> Are you using original Markdown.pl, one of the CPAN versions, or a >> non-perl version? Are you using the perl version (any of them),
> > Nope. Using Niklas Frykholm's Lua implementation: > > http://www.frykholm.se/files/markdown.lua
Cool, I've heard a lot of good things about the lua, I must play one day. Show quoted text
> Considering that the perl implementation is the 'canonical' one, > people tend to mimic its, hmmm, idiosyncrasies :)
Yep :) I maintain Text::Markdown and Text::MultiMarkdown on CPAN. I've got Text::MultiMarkdown to the point where can passes John's entire test suite + more of MultiMarkdown's test suite than the 'real' MultiMarkdown passes, and I'm planning to merge the two modules shortly.. I'm also currently trying different ways of implementing the markdown parser, as I think that it can be: a) A helluva lot quicker b) Smarter, to deal with edge cases like this and the list item example posted recently to the list. c) Pluggable - everyone and their dog wants a 'markdown like' language - but no two groups want exactly the same feature set. Rather than go the way of having a zillion 'options' (which is gonna make your code suck at best + still be inflexible), I'm trying to build a modular parser, so that people who care to do so can plug together various markdown (and/or other elements) and produce a markdown like language that works for them.. Show quoted text
> You can keep me posted.
Will do :) Cheers Tom
Download (untitled) / with headers
text/plain 496b
Speaking of speed, I did this benchmark of 3 versions. The 1.0.1 and 1.0.2b8 versions are the non-cpan perl versions of markdown. I benchmarked simply markdowning "foo". markdown 1.0.1 markdown: 3 wallclock secs ( 3.37 usr + 0.02 sys = 3.39 CPU) @ 2949.85/s (n=10000) markdown 1.0.2~b8 markdown: 6 wallclock secs ( 6.08 usr + 0.00 sys = 6.08 CPU) @ 1644.74/s (n=10000) CPAN Text::Markdown 1.0.16 markdown: 11 wallclock secs (10.48 usr + 0.01 sys = 10.49 CPU) @ 953.29/s (n=10000)
Subject: Re: [rt.cpan.org #32951] Fwd: Entity conversion glitch?
Date: Wed, 5 Mar 2008 09:03:09 +0000
To: bug-Text-Markdown [...] rt.cpan.org
From: Tomas Doran <bobtfish [...] bobtfish.net>
Download (untitled) / with headers
text/plain 434b
On 5 Mar 2008, at 01:40, Joey Hess via RT wrote: Show quoted text
> > CPAN Text::Markdown 1.0.16 > markdown: 11 wallclock secs (10.48 usr + 0.01 sys = 10.49 CPU) @ > 953.29/s (n=10000) >
From profiling Text::Markdown, *most* of the time is spent in Text::Balanced, which does the HTML Entity conversion. This was introduced in the latest version of Markdown that John wrote, to fix a number of the bugs with the HTML encoding. Cheers Tom
Closing


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.