Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Use transport_safe_name for mixin method names #2197

Open
ohmayr opened this issue Sep 25, 2024 · 0 comments
Open

Use transport_safe_name for mixin method names #2197

ohmayr opened this issue Sep 25, 2024 · 0 comments
Assignees
@ohmayr
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: cleanup An internal cleanup or hygiene concern.

Comments

@ohmayr
Copy link
Contributor

ohmayr commented Sep 25, 2024

In prep_wrapped_messages_async_method macro, update the mixin method names to use transport_safe_name similar to what we do for non-mixin methods within the same macro.

The exact code location can be looked up by searching for this issue.
@vchudnov-g vchudnov-g assigned ohmayr and unassigned parthea Sep 25, 2024
@vchudnov-g vchudnov-g added priority: p2 Moderately-important priority. Fix may not be included in next release. type: cleanup An internal cleanup or hygiene concern. labels Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ohmayr ohmayr

Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: cleanup An internal cleanup or hygiene concern.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@parthea @vchudnov-g @ohmayr