-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Modernization #2343
Modernization #2343
Conversation
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.
Except compilation issues caused by missing #include
everything else seems to be ok
I will fix the build, and may use duration_cast as suggested, and will re-request review. |
6ec5881
to
5b9e78b
Compare
As I promised. Please review the last three, new commits. Nothing has changed in the earlier commits except for rebasing and supplying needed include directives. |
f684e70
to
d162f17
Compare
d162f17
to
40af587
Compare
Requesting review of three newest commits added since last review
40af587
to
1e4a589
Compare
... where they must mix with un-typed numbers, including calls to wxTimer methods. Change the name of one so it doesn't mention the period. Do not use duration_cast, but use other conversions that check for non-lossiness at compile time.
1e4a589
to
c814e58
Compare
Use more C++14 and C++17 library types
Recommended: