This patch makes the following changes on many in-class methods.
- NS_IMETHODIMP F() override; --> NS_IMETHOD F() override;
- NS_IMETHODIMP F() override {...} --> NS_IMETHOD F() override {...}
- NS_IMETHODIMP F() final; --> NS_IMETHOD F() final;
- NS_IMETHODIMP F() final {...} --> NS_IMETHOD F() final {...}
Using NS_IMETHOD is the preferred way of marking in-class virtual methods.
Although these transformations add an explicit |virtual|, they are safe --
there's an implicit |virtual| anyway because |override| and |final| only work
with virtual methods.
--HG--
extra : rebase_source : 386ee4e4ea2ecd8d5001efabc3ac87b4d6c0659f
This removes the unnecessary setting of c-basic-offset from all
python-mode files.
This was automatically generated using
perl -pi -e 's/; *c-basic-offset: *[0-9]+//'
... on the affected files.
The bulk of these files are moz.build files but there a few others as
well.
MozReview-Commit-ID: 2pPf3DEiZqx
--HG--
extra : rebase_source : 0a7dcac80b924174a2c429b093791148ea6ac204
As part of unblocking building with VS2015u1 in automation, I'm mass
disabling compiler warnings that are turned into errors. This is not
the preferred mechanism to fix compilation warnings. So hopefully
this patch never lands because someone insists of fixing the underlying
problem instead. But if it does land, hopefully the workaround is
only temporary. That being said, the warning occurs in 3rd party ICU
code. So I'm not sure what my options are for fixing this.
MozReview-Commit-ID: 9gOSotjaQsN
--HG--
extra : rebase_source : c1a6c51bcdd4e1a882f371e7beba4f88d5c059a4
The current implementation of HierarchicalStringList allows the following:
FOO.bar = [
'foo',
'bar',
]
while
FOO.bar += [
'foo',
'bar',
]
would be invalid because of the StrictOrderingOnAppendList enforcement.
It also allows to overwrite the entire list with a subsequent
FOO.bar = [
'baz',
]
while we've explicitly forbidden such things for every other list.
While in the vicinity, fix HierarchicalStringList._get_export_variable to not
call the HierarchicalStringList constructor uselessly.
This change is for the following warning-as-error that shows up on "B2G L
Emulator" builds: "error: call to 'sprintf' declared with attribute warning:
sprintf is often misused; please use snprintf".
The patch also removes two ALLOW_COMPILER_WARNINGS=True flags, which are no
longer necessary.
--HG--
extra : rebase_source : 7fcbb06ca94b93bab4b75f6701b7f8a4838f1767
The patch removes 455 occurrences of FAIL_ON_WARNINGS from moz.build files, and
adds 78 instances of ALLOW_COMPILER_WARNINGS. About half of those 78 are in
code we control and which should be removable with a little effort.
--HG--
extra : rebase_source : 82e3387abfbd5f1471e953961d301d3d97ed2973
Create a library containing nsISocketTransportService and nsIDNS that can be used to support standalone WebRTC
--HG--
extra : rebase_source : d960247f7548cd5abb4c3ba40dd1b3c4ec5966ed
Dhanesh B. Sabane
Jonathan Kew
Xidorn Quan
Igor
Nicholas Nethercote
Tom Tromey
Xidorn Quan
Benjamin Smedberg
Chris Peterson
Gregory Szorc
Eric Rahm
Jan Beich
Mike Hommey
Ted Clancy
Carsten "Tomcat" Book
Henri Sivonen
Aniket Vyas
Simon Montagu
Makoto Kato
Kartikaya Gupta
Andy Pusch
Ted Clancy
Randall Barker
Wes Kocher