Re: [Monotone-devel] Re: RFC: workspace migration and work.cc refactor [net.venge.monotone.workspace-merge.migration]

[Derek Scherger]

Zack Weinberg wrote:
> now is that a small minority of app.work.foo() calls take an app_state
> reference and there's no rhyme nor reason to which of them it is; thus
> the thought that it might be better to put a reference to the entire
> app_state into the workspace object.  [It would not help to add more
> sub-component references, because the primary remaining use of
> app_state is to construct node_restriction objects.]

If it helps, the restrictions stuff only uses app.lua (for the
ignore_file hook) and the app.depth option setting. It doesn't touch
anything else in app_state so there's really no need to pass the whole
thing in.

Cheers,
Derek