[LyX/master] Fix bug #10635

Richard Kimberly Heck rikiheck at lyx.org
Thu Jan 14 02:50:25 UTC 2021


On 1/13/21 6:29 PM, Kornel Benko wrote:
> Am Thu, 14 Jan 2021 00:00:17 +0100
> schrieb Kornel Benko <kornel at lyx.org>:
>
>> Am Wed, 13 Jan 2021 20:38:07 +0100 (CET)
>> schrieb Richard Kimberly Heck <rikiheck at lyx.org>:
>>
>>> commit d41ec3368de969a20a16e3812bd388b4ed7beafb
>>> Author: Richard Kimberly Heck <rikiheck at lyx.org>
>>> Date:   Wed Jan 13 14:38:48 2021 -0500
>>>
>>>     Fix bug #10635
>>> ---
>>>  src/frontends/qt/GuiView.cpp |    2 ++
>>>  1 files changed, 2 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/src/frontends/qt/GuiView.cpp b/src/frontends/qt/GuiView.cpp
>>> index 714af17..07e08e0 100644
>>> --- a/src/frontends/qt/GuiView.cpp
>>> +++ b/src/frontends/qt/GuiView.cpp
>>> @@ -4162,6 +4162,8 @@ void GuiView::dispatch(FuncRequest const & cmd, DispatchResult &
>>> dr) insertLyXFile(from_utf8(cmd.getArg(0)), true);
>>>  			else
>>>  				insertLyXFile(cmd.argument());
>>> +			dr.forceBufferUpdate();
>>> +			dr.screenUpdate(Update::Force);
>>>  			break;
>>>  		}
>>>    
>> Probably? this or one of the latest commits causes crashes ATM.
>>
>> I get the attached. The workarea is the regex-field,
>> I was trying to modify the regex-content after a cancelled search
>>
>> 	Kornel
> Sorry, this commit was not the reason, the crash occurred before. I will try to bisect

No problem. Good luck!

Riki

PS I would be VERY worried if this kind of commit caused a problem.
Flagging the updates we need to do through DispatchResult is supposed to
prevent problems caused by doing such things mid-dispatch.




More information about the lyx-devel mailing list