Skip to content

#5050 Fix race condition on exit of SpeechViewer#6201

Merged
feerrenrut merged 1 commit into
masterfrom
i5050-FixRaceConditionOnExit
Aug 31, 2016
Merged

#5050 Fix race condition on exit of SpeechViewer#6201
feerrenrut merged 1 commit into
masterfrom
i5050-FixRaceConditionOnExit

Conversation

@feerrenrut

@feerrenrut feerrenrut commented Jul 22, 2016

Copy link
Copy Markdown
Contributor

When exiting nvda with the speechViewer window open, sometimes destroy is called before the queueHandler has finished speaking all text (and then calling the speechViewer). This update stops the speechViewer from being accessed after the window is destroyed.

The error raised: PyDeadObjectError: The C++ part of the SpeechViewerFrame object has been deleted, attribute access no longer allowed.

When exiting nvda with the speechViewer window open, sometimes destroy
is called before the queueHandler has finished speaking all text (and
then calling the speechViewer). This update stops the speechViewer from
being accessed after the window is destroyed.
@feerrenrut

Copy link
Copy Markdown
Contributor Author

@michaelDCurran Please take a look at this when you get the chance

@michaelDCurran

Copy link
Copy Markdown
Member

This looks fine to me.

feerrenrut added a commit that referenced this pull request Jul 29, 2016
Merge branch 'i5050-FixRaceConditionOnExit' into next
@feerrenrut feerrenrut merged commit bdd73a9 into master Aug 31, 2016
@nvaccessAuto nvaccessAuto added this to the 2016.4 milestone Aug 31, 2016
feerrenrut added a commit that referenced this pull request Aug 31, 2016
Fixed a rare issue when exiting NVDA while the speech viewer is open causing an error. (#5050)
@feerrenrut feerrenrut deleted the i5050-FixRaceConditionOnExit branch January 17, 2020 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants