>
> I would add the name of the toolbar to the error message.

Ok.

> And if the settings were written with the default value for visibility_ (0)
> then restoreSession() happily will restore that nonsense value.
>

You are right. But we should never write a value of 0.

Do you already have an idea how that can happen ?

>
> PS: I forgot to mention again the change below...
>

That's why I already committed that in r37070.

Vincent

Reply via email to