-
Notifications
You must be signed in to change notification settings - Fork 250
Const method should be thread safe #108
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
Comments
Sorry for my careless PR... It seems that one of the easiest ways to resolve this issue while maintaining Unfortunately, it seems that the huge execution time degradation occurs when Clang is used because of the overhead of mutex...
There might be a more effective way that I do not came up with to resolve this issue. At present, it seems that reverting this repo to the previous version is worth consideration (if possilble). |
OK, we decide to revert the change of TBH, I think there is no nice solution for introduce const into lazysegtree, at least we cannot accept |
By #107 , we add
const
to some methods of lazysegtree.But it seems that const method should be thread safe.
The text was updated successfully, but these errors were encountered: