On Fri, Aug 24, 2001 at 02:51:04PM +0100, Angus Leeming wrote:

> template <class Inset, class Params>
> void ControlInset<Inset, Params>::show(Params const & params)
> {
>       if (params_) delete params_;
>       params_ = new Params(params);
> 
>       setDaughterParams();
> 
> +     static bool isBuilt = false;
> +     if (!isBuilt) {
> +             isBuilt = true;
> +             view().build();
> +     }
>       bc().readOnly(isReadonly());
>       view().show();
> }

yes, it would I think. ignore my patch ;)

the static bool needs to be non-local as there are 4 places we need this
AFAICS - ControlInset twice, ControlDialogs twice

john

-- 
"That's just kitten-eating wrong."
        - Richard Henderson

Reply via email to