-
Notifications
You must be signed in to change notification settings - Fork 13.6k
[clang-tidy] modernize-use-digit-separator issue #35414 #76153
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
Open
IvanArkhipov1999
wants to merge
78
commits into
llvm:main
Choose a base branch
from
IvanArkhipov1999:modernize-use-digit-separator
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
78 commits
Select commit
Hold shift + click to select a range
3b83f86
Add empty files for use-digit-separator-check
IvanArkhipov1999 0560b3c
Finding integer value
IvanArkhipov1999 7e64853
Splitting integer string by 3 digits
IvanArkhipov1999 43f66ac
Formatting
IvanArkhipov1999 3a36679
Checking formatting of integer literal
IvanArkhipov1999 524aa8b
Messages of checking formatting integer literals
IvanArkhipov1999 78b1bf1
Support for various number systems
IvanArkhipov1999 9d4430d
Support for octal number system
IvanArkhipov1999 d80dbbe
Formatting
IvanArkhipov1999 4bd9517
Support integer literals with postfix
IvanArkhipov1999 c97e527
Begin of work with float literals
IvanArkhipov1999 544d9f8
Begin of work with float literals
IvanArkhipov1999 8115148
Fix clang tidy
IvanArkhipov1999 c9091cb
Crazy (dirty) support for float literals
IvanArkhipov1999 ceb711a
Formatting
IvanArkhipov1999 6b933a9
Getting float value string
IvanArkhipov1999 e66e180
Some refactoring
IvanArkhipov1999 0092c07
Formating
IvanArkhipov1999 e7fddc9
Support of format postfixes of floats
IvanArkhipov1999 e42ba55
Formatting
IvanArkhipov1999 28838f1
Empty test
IvanArkhipov1999 9a9fa52
Simple test
IvanArkhipov1999 9ddfb12
Add some tests
IvanArkhipov1999 04e4091
Add tests with minus
IvanArkhipov1999 9152e1e
Delete postfixes
IvanArkhipov1999 b6f82b9
Merge branch 'main' into modernize-use-digit-separator
IvanArkhipov1999 e175e70
Return postfixes
IvanArkhipov1999 c840c98
Support long postfixes
IvanArkhipov1999 43f53c5
Format
IvanArkhipov1999 977e689
Using getFormatedLiteralString
IvanArkhipov1999 68d0707
Renaming
IvanArkhipov1999 a7064ec
Create getFormatedFloatString
IvanArkhipov1999 9edf9f5
Refactor getFormatedFloatString
IvanArkhipov1999 5b4460b
Formating
IvanArkhipov1999 d59adac
Add precision calculating
IvanArkhipov1999 15627d1
Formating
IvanArkhipov1999 ba22e57
Scientific form of literal support
IvanArkhipov1999 bde206f
Some refactoring and tests
IvanArkhipov1999 8897400
Formating
IvanArkhipov1999 260e096
Fix warning message for float values
IvanArkhipov1999 808e95c
Some refactoring
IvanArkhipov1999 c22c94b
Formating
IvanArkhipov1999 b80f6a1
Support signs in exponent notation
IvanArkhipov1999 9726be5
Formating
IvanArkhipov1999 1e00d39
Some tests and fixes
IvanArkhipov1999 5081470
Formatting
IvanArkhipov1999 f929f69
Add more tests
IvanArkhipov1999 6fd7991
Add more tests
IvanArkhipov1999 78d5419
Formating
IvanArkhipov1999 fdea346
Add tests for Formatted literals
IvanArkhipov1999 f8446ab
Add tests for wrong Formatted literals
IvanArkhipov1999 11f8091
Delete tests for wrong Formatted literals
IvanArkhipov1999 f754d54
Add docs
IvanArkhipov1999 ab6dedf
Formatting
IvanArkhipov1999 1db3de8
Merge branch 'main' into modernize-use-digit-separator
IvanArkhipov1999 e31f05f
Formatting
IvanArkhipov1999 b3624ce
Formatting
IvanArkhipov1999 93f1885
Documentation
IvanArkhipov1999 004f02e
Add include sstream
IvanArkhipov1999 dd09ad2
Fix some tests
IvanArkhipov1999 91b7ff1
Remove using streams
IvanArkhipov1999 aaaac01
Formatting
IvanArkhipov1999 2ded896
Fix scientific notation bug
IvanArkhipov1999 c157fab
Formatting
IvanArkhipov1999 8bb32be
Add tests for long wrong formatted literals
IvanArkhipov1999 42a1809
Write required c++ version in docs
IvanArkhipov1999 57d8e57
Fix check fix and add tests with assignment expression.
IvanArkhipov1999 4675cfa
Formatting
IvanArkhipov1999 7253f4b
Use llvm_unreachable
IvanArkhipov1999 5c0f524
Change message for floating literals
IvanArkhipov1999 e6e2e2c
Replace std::basic_string<char> by std::string
IvanArkhipov1999 58a224d
Formatting
IvanArkhipov1999 c54ad24
Awful code for macros
IvanArkhipov1999 656b46d
Formatting
IvanArkhipov1999 fa94bb4
Add some tests and fixes
IvanArkhipov1999 c5ec616
Support binary numbers in macros
IvanArkhipov1999 a4110d8
Support oct and hex numbers in macros
IvanArkhipov1999 dcff84f
Formatting
IvanArkhipov1999 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Replace std::basic_string<char> by std::string
- Loading branch information
commit e6e2e2c5f8e5e0a2ae6aed748c6094b38371b525
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
std::accumulate isn't efficient, better simply create empty std::string, use reserve, and then append all strings in simple loop.