diff options
| author | Joey Hess <joey@kodama.kitenet.net> | 2008-03-21 04:48:26 -0400 |
|---|---|---|
| committer | Joey Hess <joey@kodama.kitenet.net> | 2008-03-21 04:48:26 -0400 |
| commit | f937c1fb8074a512d8bb788fa275f5e90595cd47 (patch) | |
| tree | 3f6b5b137c33528df10450bc95ff2829bf865622 | |
| parent | 0b9e849aba38f0695491ad5ca27de11632627ffe (diff) | |
crazy optimisation to work around slow markdown
Markdown is slow. Especially if it has to process an enormous page. The
most common enormous page is currently the recentchanges page, which gets
processed a lot, and contains very little actual markdown. Most of it is a
big <div>, which markdown skips ... slowly.
This is a rather sick optimisation to work around markdown's speed issues.
Now inline inserts a small, dummy div, allows markdown to quickly render
the actual page content, then replaces the dummy with the actual inlined
pages later.
Results: Rendering just a recentchanges page, with diffs included, dropped
from 4.5 seconds to 2.7 seconds on my laptop. Building the entire wiki
dropped from 46.6 seconds to 39.5 seconds.
(It would be better if inline were a *post*-processor directive.)
| -rw-r--r-- | IkiWiki/Plugin/inline.pm | 15 | ||||
| -rw-r--r-- | debian/changelog | 1 |
2 files changed, 15 insertions, 1 deletions
diff --git a/IkiWiki/Plugin/inline.pm b/IkiWiki/Plugin/inline.pm index c7107d664..0002371c1 100644 --- a/IkiWiki/Plugin/inline.pm +++ b/IkiWiki/Plugin/inline.pm @@ -10,6 +10,7 @@ use URI; my %knownfeeds; my %page_numfeeds; +my @inline; sub import { #{{{ hook(type => "getopt", id => "inline", call => \&getopt); @@ -19,6 +20,7 @@ sub import { #{{{ call => \&IkiWiki::preprocess_inline); hook(type => "pagetemplate", id => "inline", call => \&IkiWiki::pagetemplate_inline); + hook(type => "format", id => "inline", call => \&format); # Hook to change to do pinging since it's called late. # This ensures each page only pings once and prevents slow # pings interrupting page builds. @@ -51,6 +53,15 @@ sub checkconfig () { #{{{ } } #}}} +sub format (@) { #{{{ + my %params=@_; + + # Fill in the inline content generated earlier. This is actually an + # optimisation. + $params{content}=~s!<div class="inline" id="([^"]+)"></div>!$inline[$1]!g; + return $params{content}; +} #}}} + sub sessioncgi () { #{{{ my $q=shift; my $session=shift; @@ -304,7 +315,9 @@ sub preprocess_inline (@) { #{{{ } } - return $ret; + return $ret if $raw; + push @inline, $ret; + return "<div class=\"inline\" id=\"$#inline\"></div>\n\n"; } #}}} sub pagetemplate_inline (@) { #{{{ diff --git a/debian/changelog b/debian/changelog index 81a5d4a1d..c92cf86ed 100644 --- a/debian/changelog +++ b/debian/changelog @@ -56,6 +56,7 @@ ikiwiki (2.41) UNRELEASED; urgency=low (Old files will be automatically converted.) * Close meta tag for redir properly. * smiley: Detect smileys inside pre and code tags, and do not expand. + * inline: Crazy optimisation to work around slow markdown. -- martin f. krafft <madduck@debian.org> Sun, 02 Mar 2008 17:46:38 +0100 |