-
-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
Fix member names of AudioFrame
to match extension
#87006
Conversation
|
||
_ALWAYS_INLINE_ const float &operator[](int p_idx) const { | ||
DEV_ASSERT((unsigned int)p_idx < 2); | ||
return levels[p_idx]; |
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.
Could keep the old syntax here but nicer on release IMO and matches how we do in Vector2/3/4
4357dee
to
413bcf1
Compare
@YuriSizov only tangentially related to extensions, the alternative would be to rename the extension ones to |
@AThousandShips This is only an issue if we consider extensions to begin with, AFAIU. Which makes the GDExtension team a natural stakeholder in the solution, whether it touches extension code or not 🙂 |
True, though it's a style improvement nonetheless, though I suspect it might matter when using mixed code accessing both structures, but unsure |
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.
Might need a rebase for good measure.
413bcf1
to
d8b29ef
Compare
Thanks! |
Thank you! |
Ran without the compatibility cases and it didn't complain about any unused cases so seems that all the old uses are fixed, would need someone to confirm though
Ran CI without the old names and didn't have any problems on those platforms so should be covered