On Feb 1, 2013 5:10 PM, "Michael Hanselmann" <[email protected]> wrote:
>
> 2013/1/31 Bernardo Dal Seno <[email protected]>:
> > On 30 January 2013 18:25, Michael Hanselmann <[email protected]> wrote:
> >> I'm not sure I understand. What is this doing, and what is
> >> “_exclusive_storage” used for? Please clarify in the commit message.
> >
> > For some things I've put/changed comments inside the code. So, I've
> > appended this to the commit message:
> >   Also, it's now possible to specify the initial value of the
> >   exclusive_storage flag in the configuration file.
>
> I see you pushed this patch already. Nack, sorry. Please make
> “_EXCLUSIVE_STORAGE_KEY” a module variable, not an entry in the
> configuration directory.

I'm not sure I understand. You mean to keep the current value of the
exclusive_storage in a module variable? We store run-time information
already in cfg, so I don't see the point of having a new variable.

Bernardo

Reply via email to