From: Artem Y. <ne...@gm...> - 2008-08-18 19:57:50
|
Waylan Limberg wrote: > Artem, > > I've taken a quick glance at your code. I'll need more time to digest > it fully, but here are a few quick observations and questions: > > I'm assuming your "2.0-beta" is the latest commit on the "master" > branch, "2.1-alpha" is the "norec2" branch and the slower version (no > tarball) is the "norec" branch. Am I right on that? > Yep, that's it. "2.1-alpha" is faster, but there are some issues, in spite of all tests works fine. for instance: * item 1 * item 2 aditional text * item 3 If splitting index will point to the third line, it'll treat it as code block, not as list. For now splitting function do only forward analyzing. I'll fix it in few days. > Second, have you tested the included extensions with the no-recursion > stuff? At least the headerid and codehilite extensions could be > affected by this. > I tested headerid extension, but didn't test codehilite, because it makes only text changes. > Third, I couldn't help but notice that in the "norec" branch you undid > commit [ac578bc][]. Of course, if we're not using that one due to > performance issues, no big deal. > > [ac578bc]: http://gitorious.org/projects/python-markdown/repos/mainline/commits/ac578bc2e55183ff0d6d9002ccfdb82d38778a2b > > I think I just created this branch without pulling ac578bc to my local master branch. Yes, I don't think we'll be using it, due to performance issues. > On Mon, Aug 18, 2008 at 1:59 PM, Artem Yunusov <ne...@gm...> wrote: > >> Hello all, >> >> [2.0-beta][] and [2.1-alpha][] released. Major change in 2.1-alpha is >> fix for recursion limit bug (ticket #3), now Python-Markdown can process >> any length text. First I tried completely get rid of recursion, but that >> version was slower (about 10%) then version with recursion. In 2.1-alpha >> version input splits on parts and then every part processed, it has >> almost same performance as 2.0-beta. >> >> [2.0-beta]: http://splyer.com/markdown2.0-beta.tar.gz >> [2.1-alpha]: http://splyer.com/markdown2.1-alpha.tar.gz >> >> ------------------------------------------------------------------------- >> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge >> Build the coolest Linux based applications with Moblin SDK & win great prizes >> Grand prize is a trip for two to an Open Source event anywhere in the world >> http://moblin-contest.org/redirect.php?banner_id=100&url=/ >> _______________________________________________ >> Python-markdown-discuss mailing list >> Pyt...@li... >> https://lists.sourceforge.net/lists/listinfo/python-markdown-discuss >> >> > > > > |