Andre Poenitz wrote:
> On Mon, Mar 22, 2004 at 11:39:59AM +0000, Angus Leeming wrote:
>> Gets rid of the unused nopop_ variable and rewrites
>> Cursor::dispatch to:
>> 1. Get rid of the horrible 'operator=(safe)' call;
>> 2. Make clear that the only member variable updated is
>> Cursor::disp_:
>>
>> DispatchResult LCursor::dispatch(FuncRequest const & cmd)
>> {
>> LCursor temp_cursor(*this);
>> FuncRequest temp_cmd(cmd);
>> disp_ = dispatch_it(temp_cursor, temp_cmd);
>> return disp_;
>> }
>>
>> where dispatch_it is a function in namespace anon that contains the
>> guts of the original code.
>>
>> Ok to apply?
>
> If it makes you happy, yes.
I held off on this. LyX crashes so often and so easily that it was
impossible for me to test whether the crashes were due to my change
or not.
--
Angus
