-
Notifications
You must be signed in to change notification settings - Fork 512
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
another envy pr (no way) #5068
base: master
Are you sure you want to change the base?
another envy pr (no way) #5068
Conversation
for more information, see https://pre-commit.ci
This pull request has been automatically marked as stale because it has not had recent activity. Please address the requested changes and re-request reviews. Thank you for your contribution! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clean code 🫧
#include <bits/stdc++.h> | ||
using namespace std; | ||
|
||
struct segtree { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(optional)
don't see a well-defined c++ standard for class/struct names, but most of the solution class/struct names here use pascalcase.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
classes/structs should always use pascalcase here
Place an "x" in the corresponding checkbox if it is done or does not apply to this pull request.