Closed Bug 879054 Opened 7 years ago Closed 7 years ago

fix log printf of status field in e10s HTTP

Categories

(Core :: Networking: HTTP, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla24

People

(Reporter: jduell.mcbugs, Assigned: jduell.mcbugs)

Details

Attachments

(1 file)

Attached patch v1Splinter Review
Trial: we generally use %x for status.  (I noticed this because the %ul is typo, and should have been %lu, so it printed 'l' at the end :)
Attachment #757700 - Flags: review?(mcmanus)
s/Trial/Trivial/
Comment on attachment 757700 [details] [diff] [review]
v1

do you want to fix all the this=%x to be this=%p too? That'd be awesome! (it gets truncated on 32 bit platforms)
Attachment #757700 - Flags: review?(mcmanus) → review+
> this=%x to be this=%p 

Fixed--there were a lot of them.

  https://hg.mozilla.org/integration/mozilla-inbound/rev/a8ee9e4a2ee5
https://hg.mozilla.org/mozilla-central/rev/a8ee9e4a2ee5
Assignee: nobody → jduell.mcbugs
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.