Closed Bug 1146229 Opened 6 years ago Closed 6 years ago

Compiler warning C4229 in xpcAccessibleTable.cpp

Categories

(Core :: Disability Access APIs, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: dmajor, Assigned: dmajor)

Details

Attachments

(1 file)

Saw this scrolling past in a build. It's an interesting warning: "anachronism used : modifiers on data are ignored"

Looks like a too-eager s/nsresult/NS_IMETHODIMP/.
Attached patch patchSplinter Review
Attachment #8581422 - Flags: review?(surkov.alexander)
Comment on attachment 8581422 [details] [diff] [review]
patch

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

r=me, that was autorename issue I guess
Attachment #8581422 - Flags: review?(surkov.alexander) → review+
https://hg.mozilla.org/mozilla-central/rev/c4fe64d3a1c5
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Assignee: nobody → dmajor
You need to log in before you can comment on or make changes to this bug.