enum class fixups - testing non-zero

services/files/shared_impl.cc

Index: services/files/shared_impl.cc
diff --git a/services/files/shared_impl.cc b/services/files/shared_impl.cc
index 8584f82d108121c42c2499e916e5b29653137834..24cdc3be642f8fd51a65c0534edaf2006a3f68b7 100644
--- a/services/files/shared_impl.cc
+++ b/services/files/shared_impl.cc
@@ -59,12 +59,15 @@ void TouchFD(int fd,
              const TouchFDCallback& callback) {
   DCHECK_NE(fd, -1);
 
+  Error error;
   struct timespec times[2];
-  if (Error error = TimespecOrNowToStandardTimespec(atime.get(), &times[0])) {
+  if ((error = TimespecOrNowToStandardTimespec(atime.get(), &times[0]))
+      != ERROR_OK) {

[viettrungluu, 2015/09/22 22:04:58]

You should also "git cl format" the change. The standard formatting puts the !=
on the previous line.

[johngro, 2015/09/22 22:24:12]

Done.

     callback.Run(error);
     return;
   }
-  if (Error error = TimespecOrNowToStandardTimespec(mtime.get(), &times[1])) {
+  if ((error = TimespecOrNowToStandardTimespec(mtime.get(), &times[1]))
+      != ERROR_OK) {
     callback.Run(error);
     return;
   }