commit 990852ff7dd5728b844d7dfd36552834e1ae1a87
author Tom Cherry <tomcherry@google.com> Wed Oct 02 10:34:03 2019 -0700
committer Tom Cherry <tomcherry@google.com> Wed Oct 02 14:27:04 2019 -0700
tree df732fdf8fe77dd03c5e234b7068b8e0f01d3fd3
parent 97ec4eecd44f04d6c6580846c1efe9acb75bb06b

liblog: remove superfluous checks

More checks in static functions where all callers always satisfy them.

Test: build
Change-Id: I3c9bd1fd4e5c55a2f9f2c42d5259c5cdd8f8a853

liblog/fake_writer.cpp

diff --git a/liblog/fake_writer.cpp b/liblog/fake_writer.cpp
index 4d07caa..f1ddff1 100644
--- a/liblog/fake_writer.cpp
+++ b/liblog/fake_writer.cpp
@@ -24,6 +24,7 @@
 #include "log_portability.h"
 #include "logger.h"
 
+static int fakeAvailable(log_id_t);
 static int fakeOpen();
 static void fakeClose();
 static int fakeWrite(log_id_t log_id, struct timespec* ts, struct iovec* vec, size_t nr);
@@ -34,12 +35,16 @@
     .name = "fake",
     .logMask = 0,
     .context.priv = &logFds,
-    .available = NULL,
+    .available = fakeAvailable,
     .open = fakeOpen,
     .close = fakeClose,
     .write = fakeWrite,
 };
 
+static int fakeAvailable(log_id_t) {
+  return 0;
+}
+
 static int fakeOpen() {
   int i;