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.

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

[onert] Apply inline variable to all other applicable locations #14653

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ragmani
Copy link
Contributor

@ragmani ragmani commented Feb 12, 2025

This commit applies inline variable to all other applicable locations.

ONE-DCO-1.0-Signed-off-by: ragmani [email protected]

This commit applies inline variable to all other applicable locations.

ONE-DCO-1.0-Signed-off-by: ragmani <[email protected]>
@ragmani ragmani added the PR/ready for review It is ready to review. Please review it. label Feb 12, 2025
@ragmani ragmani requested a review from a team February 12, 2025 11:33
Comment on lines +72 to +73
static inline std::mutex _session_id_mutex;
static inline uint32_t _next_session_id = 0;
Copy link
Contributor

Choose a reason for hiding this comment

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

Same question with #14652 (comment)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Same answer with #14652 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR/ready for review It is ready to review. Please review it.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants