enum class fixups - testing non-zero

examples/echo_terminal/main.cc

Index: examples/echo_terminal/main.cc
diff --git a/examples/echo_terminal/main.cc b/examples/echo_terminal/main.cc
index 548a463aff46151c505972208205d71eb173361a..333b5c7cca4c4be41f729a9b0bc87dac1c374e2c 100644
--- a/examples/echo_terminal/main.cc
+++ b/examples/echo_terminal/main.cc
@@ -41,7 +41,7 @@ class TerminalEchoer {
 
   // |Read()| callback:
   void OnRead(mojo::files::Error error, mojo::Array<uint8_t> bytes_read) {
-    if (error) {
+    if (mojo::files::ERROR_OK != error) {

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

In general, I (and most humans) prefer to read "error != mojo::files::ERROR_OK".

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

Done.

       LOG(ERROR) << "Error: Read(): " << error;
       delete this;
       return;
@@ -71,7 +71,7 @@ class TerminalEchoer {
 
   // |Write()| callback:
   void OnWrite(mojo::files::Error error, uint32_t num_bytes_written) {
-    if (error) {
+    if (mojo::files::ERROR_OK != error) {
       LOG(ERROR) << "Error: Write(): " << error;
       delete this;
       return;