code review
authorJoey Hess <joey@kitenet.net>
Mon, 5 Sep 2011 17:49:37 +0000 (13:49 -0400)
committerJoey Hess <joey@kitenet.net>
Mon, 5 Sep 2011 17:49:37 +0000 (13:49 -0400)
doc/todo/allow_site-wide_meta_definitions.mdwn

index a308e0d8c5c913ba331ca8146b3e82e2e972b177..d1034bf589c59111a25be250c3b7b4cd19233acf 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,8 @@ 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]]