-
-
Notifications
You must be signed in to change notification settings - Fork 957
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
Codechange: replace left QSortT() with std::sort() #7509
Conversation
Using |
And of course OSX still uses QSortT() |
Used the easy way for OSX as I can't test it myself. |
8f39b88
to
081f6fa
Compare
several of these std::arrays feel like they could be replaced with std::vector, and the _foobar_size globals could be removed as a result Also, std::begin() & std::end() can be used on C-style arrays |
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.
As discussed - a few more usages of std::vector
please :)
56f4526
to
e2bf7dc
Compare
d262609
to
739edaa
Compare
To sort arrays are replaced with std::array or std::vector depending on context.