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
Investigate changing MessageDrop::runEvent to std::string_view #28451
Comments
assign core |
New categories assigned: core @Dr15Jones,@smuzaffar you have been requested to review this Pull request/Issue and eventually sign? Thanks |
A new Issue was created by @Dr15Jones Chris Jones. @davidlange6, @Dr15Jones, @smuzaffar, @fabiocos, @kpedro88 can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
See #28457 for a solution |
@Dr15Jones your #28457 has been now integrated |
+1 |
This issue is fully signed and ready to be closed. |
With the change #28450 it looks like all the string copies done to fill
MessageDrop::runEvent
are now redundant as the source string has a lifetime as long as that value ofrunEvent
is in use.The text was updated successfully, but these errors were encountered: