Skip to content

Improve comment in extract_windows system about main thread window handling#5071

Closed
alice-i-cecile wants to merge 2 commits intobevyengine:mainfrom
alice-i-cecile:patch-4
Closed

Improve comment in extract_windows system about main thread window handling#5071
alice-i-cecile wants to merge 2 commits intobevyengine:mainfrom
alice-i-cecile:patch-4

Conversation

@alice-i-cecile
Copy link
Copy Markdown
Member

This is useful to give some indication that the _marker parameter cannot be safely removed.

@alice-i-cecile alice-i-cecile added A-Rendering Drawing game state to the screen C-Code-Quality A section of code that is hard to understand or change labels Jun 22, 2022
@alice-i-cecile
Copy link
Copy Markdown
Member Author

bors try

bors bot added a commit that referenced this pull request Jun 22, 2022
@bors
Copy link
Copy Markdown
Contributor

bors bot commented Jun 22, 2022

try

Build failed:

Co-authored-by: Daniel McNab <36049421+DJMcNab@users.noreply.github.com>
@alice-i-cecile alice-i-cecile changed the title Add missing SAFETY comment to extract_windows system Improve safety comment in extract_windows system Jun 22, 2022
@alice-i-cecile alice-i-cecile changed the title Improve safety comment in extract_windows system Improve comment in extract_windows system about main thread window handling Jun 22, 2022
Copy link
Copy Markdown
Member

@DJMcNab DJMcNab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean sure, I guess.

I'd be just as happy to close this out, since this is already a few lines above

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Rendering Drawing game state to the screen C-Code-Quality A section of code that is hard to understand or change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants