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.

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

StrFormat() is a printf-like function, mark it as such, fix fallout. #727

Merged
merged 1 commit into from
Nov 27, 2018

Conversation

LebedevRI
Copy link
Collaborator

Fixes #714.

@AppVeyorBot
Copy link

Build benchmark 1573 failed (commit d342a80563 by @LebedevRI)

@coveralls
Copy link

coveralls commented Nov 26, 2018

Coverage Status

Coverage increased (+0.006%) to 89.294% when pulling 1ad81ce on LebedevRI:print-range into c9311a4 on google:master.

@AppVeyorBot
Copy link

Build benchmark 1574 failed (commit c9cd2e6927 by @LebedevRI)

@LebedevRI LebedevRI force-pushed the print-range branch 2 times, most recently from f431fcd to b7387f0 Compare November 26, 2018 17:08
@AppVeyorBot
Copy link

Build benchmark 1576 failed (commit ed94bded4a by @LebedevRI)

@AppVeyorBot
Copy link

Build benchmark 1578 failed (commit b12d0c87e3 by @LebedevRI)

@AppVeyorBot
Copy link

Build benchmark 1579 completed (commit 67c53f8cf4 by @LebedevRI)

@AppVeyorBot
Copy link

Build benchmark 1580 completed (commit 376b12b007 by @LebedevRI)

@EricWF EricWF merged commit c9f2693 into google:master Nov 27, 2018
@EricWF
Copy link
Contributor

EricWF commented Nov 27, 2018

LGTM. Thanks.

@LebedevRI LebedevRI deleted the print-range branch November 27, 2018 05:58
@LebedevRI
Copy link
Collaborator Author

Thank you for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants