]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blobdiff - doc/todo/allow_site-wide_meta_definitions.mdwn
review
[git.ikiwiki.info.git] / doc / todo / allow_site-wide_meta_definitions.mdwn
index a308e0d8c5c913ba331ca8146b3e82e2e972b177..f548f1a5b45c2dc4d8c809efc921eb1e641b5131 100644 (file)
@@ -83,6 +83,8 @@ definitions essentially.
 >>
 >>> I generally agree with this. It is *possible* that meta would have a new
 >>> field added, that takes parameters and make sense to use globally.
+>>> (Indeed, this later happened to some extent with the sortas parameters
+>>> being added to some metas.)
 >>> --[[Joey]] 
 >>
 >> Due to this, and the added complexity of the second patch (having to adjust
@@ -150,3 +152,18 @@ definitions essentially.
 >>>>> 
 >>>>> The patch can be found at the git branch described above.
 >>>>>  — [[Jon]]
+
+>>>>>> I wish I could take pity on you and just merge this, but 
+>>>>>> AFAICS it still suffers from the memory bloat described above.
+>>>>>> Specifically, when `scan` calls `preprocess`, it
+>>>>>> stores the metadata in `%pagestate` etc. --[[Joey]]
+
+>>>>>>> No pity required — but whoops, yes, that was a bit of a mistake
+>>>>>>> ☺ I guess I'll have to split the current `preprocess` in half.
+>>>>>>> — [[Jon]]
+
+>>>>>>>> I've been taking another look at this today, as I'm very keen to
+>>>>>>>> close various open loops of mine in IkiWiki to move on and do some
+>>>>>>>> other stuff.  However, I'm not actually *using* this at the moment,
+>>>>>>>> so whilst I think it's a good idea, I can't really motivate myself
+>>>>>>>> to fix it anymore. I guess for now, this should just rot. — [[Jon]]