argument name 'reuse' in comment does not match parameter name 'addressReuse'

RESOLVED FIXED in Firefox 60

Status

()

defect
P4
normal
Rank:
35
RESOLVED FIXED
2 years ago
Last year

People

(Reporter: sylvestre, Assigned: spencerb, Mentored)

Tracking

({good-first-bug})

Trunk
mozilla60
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox60 fixed)

Details

(Whiteboard: [good first bug][lang=C++])

Attachments

(1 attachment, 1 obsolete attachment)

Assignee: nobody → spencerb
Reporter

Updated

2 years ago
Attachment #8926887 - Attachment is obsolete: true
Assignee

Comment 3

2 years ago
Thanks.  First attachment was an error due to my inexperience with mercurial.
Assignee

Updated

2 years ago
Attachment #8926897 - Flags: review?(sledru)
Reporter

Comment 4

2 years ago
Comment on attachment 8926897 [details] [diff] [review]
Change commented argument name to match declaration

Good, please try to find an official reviewer for this code!
Attachment #8926897 - Flags: review?(sledru) → feedback+
Assignee

Updated

2 years ago
Attachment #8926897 - Flags: review?(drno)

Updated

2 years ago
Rank: 35
Priority: -- → P4
Comment on attachment 8926897 [details] [diff] [review]
Change commented argument name to match declaration

Review of attachment 8926897 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM
And Thanks!
Attachment #8926897 - Flags: review?(drno) → review+

Comment 6

Last year
Hey, is this issue still open?

Comment 8

Last year
Pushed by sledru@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/09894bd08b93
Change commented argument name to match declaration r=drno
nope, sorry!

Comment 10

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/09894bd08b93
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in before you can comment on or make changes to this bug.