New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Image overlay bugs #246
Image overlay bugs #246
Conversation
88b2233
to
0eed1fe
Compare
@EvidentlyCube , the other PRs are merged. This PR can now be rebased. |
…t duplicate the effect for one turn
… it's closed. Everything also works correctly with losing and regaining focus. Finally, dialogs will delegate handling windows focus back to their parent (who should either be a screen or another dialog, which in turn will delegate it all the way back to a screen). This allows having special handling for focus, eg. repausing sound if dialog is open
0eed1fe
to
b6be6ad
Compare
@mrimer Ready to review and merge |
{ | ||
g_pTheSound->PauseSounds(); | ||
|
||
const Uint32 dwSpeechRemaining = this->dwNextSpeech - SDL_GetTicks(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I note here that if dwNextSpeech == 0, then this unsigned value will translate the negative ts to a large positive.
That seems benign, though, as the value won't be used in this case.
@EvidentlyCube , Really elegant improvements built on your nice effect refactoring. I like the vision you put forth and executed on! |
Marked as draft since it depends on changes from #228 and #245.
Relevant thread