summaryrefslogtreecommitdiff
path: root/doc
diff options
context:
space:
mode:
authorJoey Hess <joey@gnu.kitenet.net>2009-08-27 21:36:55 -0400
committerJoey Hess <joey@gnu.kitenet.net>2009-08-27 21:36:55 -0400
commitb7a3fbec78a7584f24b8dcc38e5e9537ad95c7f5 (patch)
tree39cd24a069edca4065ea23998a8c5bef8ae4065e /doc
parent04a8da4f0176d153fed7d9a1c483f19b70f32900 (diff)
response
Diffstat (limited to 'doc')
-rw-r--r--doc/todo/optimize_simple_dependencies.mdwn20
1 files changed, 20 insertions, 0 deletions
diff --git a/doc/todo/optimize_simple_dependencies.mdwn b/doc/todo/optimize_simple_dependencies.mdwn
index c3ccd5ad0..44163311b 100644
--- a/doc/todo/optimize_simple_dependencies.mdwn
+++ b/doc/todo/optimize_simple_dependencies.mdwn
@@ -46,3 +46,23 @@ beyond the ready/depends-exact branch point, so the results should be
equally valid.)
--[[smcv]]
+
+> We discussed this on irc; I had some worries that things may have been
+> switched to `add_depends_exact` that were not pure page names. My current
+> feeling is it's all safe, but who knows. It's easy to miss something.
+> Which makes me think this is not a good interface.
+>
+> Why not, instead, make `add_depends` smart. If it's passed something
+> that is clearly a raw page name, it can add it to the exact depends hash.
+> Else, add it to the pagespec hash. You can tell if it's a pure page name
+> by matching on `$config{wiki_file_regexp}`.
+>
+> Also I think there may be little optimisation value left in
+> 7227c2debfeef94b35f7d81f42900aa01820caa3, since the "regular" dependency
+> lists will be much shorter.
+>
+> Sounds like inline pagenames has an already exstant bug WRT
+> pages moving, which this should not make worse. Would be good to verify.
+>
+> Re coding, it would be nice if `refresh()` could avoid duplicating
+> the debug message, etc in the two cases. --[[Joey]]