gecko-dev/hal/gonk
Nicholas Nethercote 34dcc7b852 Bug 1299384 - Use MOZ_MUST_USE with NS_warn_if_impl(). r=erahm.
This change avoids lots of false positives for Coverity's CHECKED_RETURN
warning, caused by NS_WARN_IF's current use in both statement-style and
expression-style.

In the case where the code within the NS_WARN_IF has side-effects, I made the
following change.

> NS_WARN_IF(NS_FAILED(FunctionWithSideEffects()));
> -->
> Unused << NS_WARN_IF(NS_FAILED(FunctionWithSideEffects()));

In the case where the code within the NS_WARN_IF lacks side-effects, I made the
following change.

> NS_WARN_IF(!condWithoutSideEffects);
> -->
> NS_WARNING_ASSERTION(condWithoutSideEffects, "msg");

This has two improvements.
- The condition is not evaluated in non-debug builds.
- The sense of the condition is inverted to the familiar "this condition should
  be true" sense used in assertions.

A common variation on the side-effect-free case is the following.

> nsresult rv = Fn();
> NS_WARN_IF_(NS_FAILED(rv));
> -->
> DebugOnly<nsresult rv> = Fn();
> NS_WARNING_ASSERTION(NS_SUCCEEDED(rv), "Fn failed");

--HG--
extra : rebase_source : 58788245021096efa8372a9dc1d597a611d45611
2016-09-02 17:12:24 +10:00
..
fanotify.h
GonkDiskSpaceWatcher.cpp Bug 1293603 (part 2) - Make Run() declarations consistent. r=erahm. 2016-08-08 12:18:10 +10:00
GonkFMRadio.cpp Bug 1293603 (part 2) - Make Run() declarations consistent. r=erahm. 2016-08-08 12:18:10 +10:00
GonkHal.cpp Bug 1293603 (part 2) - Make Run() declarations consistent. r=erahm. 2016-08-08 12:18:10 +10:00
GonkSensor.cpp Bug 1293603 (part 2) - Make Run() declarations consistent. r=erahm. 2016-08-08 12:18:10 +10:00
GonkSensorsHelpers.cpp Bug 1194721: Add helpers for Gonk sensors daemon, r=gsvelto 2016-02-04 12:35:13 +01:00
GonkSensorsHelpers.h Bug 1194721: Add helpers for Gonk sensors daemon, r=gsvelto 2016-02-04 12:35:13 +01:00
GonkSensorsInterface.cpp Bug 1260672: Use |UniquePtr| for Gonk sensors, r=gsvelto 2016-03-30 13:18:00 +02:00
GonkSensorsInterface.h Bug 1260672: Use |UniquePtr| for Gonk sensors, r=gsvelto 2016-03-30 13:18:00 +02:00
GonkSensorsPollInterface.cpp Bug 1260672: Use |UniquePtr| for Gonk sensors, r=gsvelto 2016-03-30 13:18:00 +02:00
GonkSensorsPollInterface.h Bug 1260672: Use |UniquePtr| for Gonk sensors, r=gsvelto 2016-03-30 13:18:00 +02:00
GonkSensorsRegistryInterface.cpp Bug 1260672: Use |UniquePtr| for Gonk sensors, r=gsvelto 2016-03-30 13:18:00 +02:00
GonkSensorsRegistryInterface.h Bug 1260672: Use |UniquePtr| for Gonk sensors, r=gsvelto 2016-03-30 13:18:00 +02:00
GonkSwitch.cpp Bug 1293603 (part 2) - Make Run() declarations consistent. r=erahm. 2016-08-08 12:18:10 +10:00
nsIRecoveryService.idl
SensorsTypes.h Bug 1194721: Add helpers for Gonk sensors daemon, r=gsvelto 2016-02-04 12:35:13 +01:00
SystemService.cpp Bug 1299384 - Use MOZ_MUST_USE with NS_warn_if_impl(). r=erahm. 2016-09-02 17:12:24 +10:00
tavarua.h
UeventPoller.cpp Bug 1273520 - NS_NewRunnable* renaming and include task.h r=fabrice 2016-05-18 18:25:35 +02:00
UeventPoller.h