-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Remove unnecessary cast from endpoint-config. #24848
Remove unnecessary cast from endpoint-config. #24848
Conversation
The cast would have just covered up issues if any came up. This way we know our types match up.
PR #24848: Size comparison from 87e7510 to 634e82b Increases (3 builds for bl702, cc13x2_26x2, psoc6)
Decreases (7 builds for bl602, cc13x2_26x2, esp32, psoc6, telink)
Full report (53 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
@bzbarsky-apple - needs regen? |
Also seeing this: this piece of code does not depend on template input, so maybe it should be moved to separate files (although I could see that having a separate macro file would also be annoying) |
Ah, yes, for the test expectations.
Yes, we could try to split up endpoint_config in various ways. But the split out bits would be pretty tighly coupled to the generated code, so it would be a bit hard to follow... |
PR #24848: Size comparison from 87e7510 to 8b8043d Increases (3 builds for bl702, cc13x2_26x2, telink)
Decreases (6 builds for bl602, cc13x2_26x2, esp32, psoc6)
Full report (53 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
* Remove unnecessary cast from endpoint-config. The cast would have just covered up issues if any came up. This way we know our types match up. * Regenerate generated code.
The cast would have just covered up issues if any came up. This way we know our types match up.