690235 - fix mac build with clang

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

Attachment: fix the clang build

+++ This bug was initially created as a clone of Bug #689397 +++

This is a small part of 689397 that can be committed earlier and fixes the clang build.

Comment 1

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

Try at
https://tbpl.mozilla.org/?tree=Try&rev=84599c41c8c4

Comment 2

[(no longer active)]

Comment on attachment 563303 [details] [diff] [review]
fix the clang build

r=me.

Comment 3

[(no longer active)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ac24c47e2885

Comment 4

[(no longer active)]

Comment on attachment 563303 [details] [diff] [review]
fix the clang build

This is a very low risk patch, which effectively doesn't change any semantics, but enables Aurora to be built using clang.

Comment 5

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/f96e4ff817a7

I accidentally pushed this patch to mozilla-central while it was living on inbound.  On the next merge, it will be merged.  Sorry for the mess!

Comment 6

[(no longer active)]

Merged from inbound: https://hg.mozilla.org/mozilla-central/rev/ac24c47e2885

Comment 7

[(no longer active)]

http://hg.mozilla.org/releases/mozilla-aurora/rev/485f3c637b11

Comment 8

[u279076]

Is there something QA can do to verify this fix?

Comment 9

[Rafael Ávila de Espíndola (:espindola) (not reading bugmail)]

(In reply to Anthony Hughes, Mozilla QA (irc: ashughes) from comment #8)
> Is there something QA can do to verify this fix?

I don't think so, it was just a build fix.