review

[git.ikiwiki.info.git] / doc / todo / allow_site-wide_meta_definitions.mdwn

diff --git a/doc/todo/allow_site-wide_meta_definitions.mdwn b/doc/todo/allow_site-wide_meta_definitions.mdwn
index c0da9323b65a74bc9c5df03e41f9e52b3a39b42a..f548f1a5b45c2dc4d8c809efc921eb1e641b5131 100644 (file)
--- a/doc/todo/allow_site-wide_meta_definitions.mdwn
+++ b/doc/todo/allow_site-wide_meta_definitions.mdwn
@@ -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
@@ -148,6 +150,20 @@ definitions essentially.
 >>>>> array-of-hashes, which means this is suitable for exposing via websetup.
 >>>>> `getsetup()` has been adjusted accordingly.
 >>>>> 
->>>>> The patch can be found at
->>>>> [[!template id=gitbranch branch=jmtd/defaultmeta author="[[jon]]"]]
+>>>>> 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]]