Comment 3 for bug 2028087
Revision history for this message
| p.w.wong (phil-weballey) wrote Re: [Bug 2028087] Re: Creating Mask crash fix | #3 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
(Get the code!)
On 7/25/23 1:42 PM, tmodes wrote:
> Adding twice a comment with your initials and a date is not helpful. This can be tracked with mercurial.
> More helpful would be a comment with a description why the separate code block is necessary.
>
> And why does it only happens with exclude region? Should be the same
> happen also with include region?
>
> ** Changed in: hugin
> Status: New => Incomplete
>
Sorry about the comment... it was for my internal use and I forget to
remove.
As for the separate code block, from stepping through with the debugger,
there appears to be a race condition that appears to resolved by
serializing in this fashion. If I recall correctly, I was getting a
fault in the m_overlay.Reset() before the overlaydc.Clear() completed.
-phil
--
"Pretending that the all-but-inevitable is inconceivable doesn't
make it impossible."
Phillip W. Wong, PE (<email address hidden>)