Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Core][BuiltinTimer] add ostream operator #11637

Merged
merged 5 commits into from
Sep 30, 2023
Merged

[Core][BuiltinTimer] add ostream operator #11637

merged 5 commits into from
Sep 30, 2023

Conversation

philbucher
Copy link
Member

@philbucher philbucher commented Sep 30, 2023 •
edited

now we can directly print the timer to print the time in a readably way

examples:

  • 15.22s => 15.22 [s]
  • 75.22s => 1 [m] 15 [s]
  • 3675.22s => 1 [h] 1 [m] 15 [s]

godbolt link

@philbucher philbucher requested a review from a team as a code owner September 30, 2023 13:43
@philbucher philbucher added Kratos Core FastPR This Pr is simple and / or has been already tested and the revision should be fast Usability labels Sep 30, 2023
@philbucher philbucher requested review from loumalouomega and removed request for a team September 30, 2023 13:44
@philbucher philbucher merged commit b652b8e into master Sep 30, 2023
11 checks passed
@philbucher philbucher deleted the core/timer-ostream branch September 30, 2023 23:16
@philbucher philbucher mentioned this pull request Oct 1, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@loumalouomega loumalouomega loumalouomega approved these changes

Assignees
No one assigned
Labels
FastPR This Pr is simple and / or has been already tested and the revision should be fast Kratos Core Usability
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@philbucher @loumalouomega