Revert "refactor the exit of nvda and gui.terminate"

[seanbudd]

Reverts #12286

[AppVeyorBot]

• PR introduces Flake8 errors
• Build (for testing PR)

See test results for failed build of commit 57dc902eba

[Brian1Gaff]

You are aware that the current alpha as of yesterday is seriously broken, and will not run sensibly on anything I suppose? Brian ***@***.*** Sent via blueyonder. Please address personal E-mail to:- ***@***.***, putting 'Brian Gaff' in the display name field. Newsgroup monitored: alt.comp.blind-users ----- Original Message ----- From: "Michael Curran" ***@***.***> To: "nvaccess/nvda" ***@***.***> Cc: "Subscribed" ***@***.***> Sent: Friday, April 23, 2021 4:56 AM Subject: Re: [nvaccess/nvda] Revert "refactor the exit of nvda and gui.terminate" (#12326)

…

Merged #12326 into master. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: #12326 (comment)

[seanbudd]

You are aware that the current alpha as of yesterday is seriously broken, and will not run sensibly on anything I suppose? Brian @.*** Sent via blueyonder. Please address personal E-mail to:- @., putting 'Brian Gaff' in the display name field. Newsgroup monitored: alt.comp.blind-users ----- Original Message ----- From: "Michael Curran" @.> To: "nvaccess/nvda" @.> Cc: "Subscribed" @.> Sent: Friday, April 23, 2021 4:56 AM Subject: Re: [nvaccess/nvda] Revert "refactor the exit of nvda and gui.terminate" (#12326)
…
Merged #12326 into master. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: #12326 (comment)

Just to confirm, you are referring to the latest alpha alpha-22498,d9afe35d, which includes this PR that has reverted the broken change?

[CyrilleB79]

@seanbudd, @michaelDCurran:
Since #12286 was reverted through this PR, shouldn't #12238 be reopened?

[seanbudd]

Yes, thanks @CyrilleB79