On Tue, 2011-01-11, Julian Foad wrote: > The complete patch (with log message) is attached. Any review would be > appreciated.
To be more specific: I'm happy with the way this patch uses triggers to inc and dec the ref counts. Review of impl details here would be appreciated. I'm not 100% sure whether close_wcroot() is the best place to delete unreferenced pristines. Review of the concept would be useful here, in comparison with other options such as deleting after flushing the work queue or at some other place. In any case, I think this is good enough (modulo the debugging code as identified by ### in the log message) to commit tomorrow and improve afterwards. - Julian
