[Gnucash-changes] r14495 - gnucash/trunk/src/register/ledger-core - Avoid crash from opening transaction in more than one register. Bug #347089.

[Derek Atkins]

Chris Shoemaker <c.shoemaker at cox.net> writes:

>> Are you sure you want to xaccTransCommitEdit() without being inside
>> suspend_gui_refresh()?  
>
> I don't see why not.

It could cause unpleasant visual aspects...

>> I would think that this might cause strange behaviors.  
>
> It might.  What kind of strange behavior did you have in mind?

I don't know.  ISTR Andreas reporting an issue on IRC.  Maybe he
can respond here?

>> Or was that your goal?
>
> No, the goal there was to ensure that the function doesn't return
> after the suspend but before the resume.

Okay.  Let me ponder that a bit.  Perhaps just adding a "resume
events" just before the return?  I'll propose an alternate patch
later.

> -chris

-derek

-- 
       Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
       Member, MIT Student Information Processing Board  (SIPB)
       URL: http://web.mit.edu/warlord/    PP-ASEL-IA     N1NWH
       warlord at MIT.EDU                        PGP key available