DescriptionReland of Fix formatting specifier for MSVC++ 2013 compatibility. (patchset #1 id:1 of https://codereview.chromium.org/1414883010/ )
Reason for revert:
Per off-line conversation, this wasn't the cause of the memory bot failures. Back into the tree you go! :)
Original issue's description:
> Revert of Fix formatting specifier for MSVC++ 2013 compatibility. (patchset #1 id:1 of https://codereview.chromium.org/1411663012/ )
>
> Reason for revert:
> Speculatively revert to try to fix this:
>
> http://build.chromium.org/p/chromium.memory.fyi/builders/Windows%20Unit%20(DrMemory%20full)%20(2)
>
> The error message is very obtuse, but seems to indicate it's some kind of string formatting issue. Feel free to revert the revert if it doesn't fix the issue.
>
> Tommy
>
> Original issue's description:
> > Fix formatting specifier for MSVC++ 2013 compatibility.
> >
> > The use of %zu in a printf format string is non-standard. This change
> > is a follow-up to https://codereview.chromium.org/1417873007 to use the
> > standard PRIuS macro instead.
> >
> > BUG=519022
> > TBR=brucedawson@chromium.org
> >
> > Committed: https://crrev.com/1dbefc62bf1df69475bea39c5a41f7825c72a6b0
> > Cr-Commit-Position: refs/heads/master@{#357582}
>
> TBR=brucedawson@chromium.org,miu@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=519022
>
> Committed: https://crrev.com/2054dbed4e6ade0129716eeffdcd9f2e4b34fd1c
> Cr-Commit-Position: refs/heads/master@{#357956}
TBR=brucedawson@chromium.org,tommycli@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=519022
Committed: https://crrev.com/4f39e75faf07800d60c76616716040f35cefd868
Cr-Commit-Position: refs/heads/master@{#358125}
Patch Set 1 #
Messages
Total messages: 4 (0 generated)
|