diff options
author | smcv <smcv@web> | 2015-06-08 19:07:46 -0400 |
---|---|---|
committer | admin <admin@branchable.com> | 2015-06-08 19:07:46 -0400 |
commit | 971dfdd6869c9729d7d52e07af695af54d91b546 (patch) | |
tree | 96c646edca1c1a611d5757596dbe3ce13aa9f368 /doc/bugs/template_creation_error.mdwn | |
parent | 9d20373c2529ddbe7a6357ef88a7ba2794f14859 (diff) | |
download | ikiwiki-971dfdd6869c9729d7d52e07af695af54d91b546.tar ikiwiki-971dfdd6869c9729d7d52e07af695af54d91b546.tar.gz |
de-optimized to fix a bug
Diffstat (limited to 'doc/bugs/template_creation_error.mdwn')
-rw-r--r-- | doc/bugs/template_creation_error.mdwn | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/doc/bugs/template_creation_error.mdwn b/doc/bugs/template_creation_error.mdwn index 33a863ec5..81b8a5c58 100644 --- a/doc/bugs/template_creation_error.mdwn +++ b/doc/bugs/template_creation_error.mdwn @@ -234,6 +234,13 @@ same logic as IkiWiki itself. I don't think that's serious. --[[smcv]] >>> drop the contents of `%scanned` and rely on a single boolean >>> flag instead. >>> +>>>> This is not actually good enough for the templatebody +>>>> directive, which does in fact need to scan certain pages +>>>> during the render phase, namely when a page that is rendered +>>>> due to dependencies uses a template that no other page being +>>>> rendered in this pass was using. I've reverted this optimization, +>>>> to fix [[wrong rendering of templatebody]]. --[[smcv]] +>>> >>> `%scanned` is likely to be no larger than `%rendered`, which >>> we already track, and whose useful lifetime does not overlap >>> with `%scanned` now. I was tempted to merge them both and call |