Skip to content

Conversation

@bruvzg
Copy link
Member

@bruvzg bruvzg commented Apr 14, 2025

- Remove outdated #pragma once note, it is now preferred style.

  • Changes auto notes to reflect actual state of the codebase (it is used a lot in two mentioned scenarios).

@bruvzg bruvzg added area:engine details Issues and PRs related to the Engine Details section of the documentation topic:codestyle labels Apr 14, 2025
Copy link
Contributor

@skyace65 skyace65 left a comment

Choose a reason for hiding this comment

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

There's an existing PR open for pragma once that covers more #10756. Any review on that would be appreciated,

Other than the one issue I found the auto keyword stuff looks good.

@bruvzg bruvzg changed the title Remove outdated #pragma once note, clarify auto usage. Clarify auto usage. Apr 15, 2025
@bruvzg
Copy link
Member Author

bruvzg commented Apr 15, 2025

There's an existing PR open for pragma once that covers more #10756.

Reverted this part.

@skyace65 skyace65 merged commit cd06455 into godotengine:master Apr 15, 2025
1 check passed
@skyace65
Copy link
Contributor

Thanks!

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

Labels

area:engine details Issues and PRs related to the Engine Details section of the documentation cherrypick:4.4 enhancement topic:codestyle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants