log: remove unnecessary stubs
authorStephen Hemminger <stephen@networkplumber.org>
Tue, 9 Dec 2014 16:35:46 +0000 (08:35 -0800)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Thu, 15 Jan 2015 12:41:39 +0000 (13:41 +0100)
The read/seek/close stub functions are unnecessary on the
log stream.  Per glibc fopencookie man page:

       cookie_read_function_t *read
              If *read is a null pointer, then reads from  the  custom  stream
              always return end of file.

       cookie_seek_function_t *seek
              If *seek is a null pointer, then it is not possible  to  perform
              seek operations on the stream.

       cookie_close_function_t *close
              If  *close is NULL, then no special action is performed when the
              stream is closed.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
lib/librte_eal/linuxapp/eal/eal_log.c

index 94dedfb..a2d9056 100644 (file)
@@ -83,33 +83,8 @@ console_log_write(__attribute__((unused)) void *c, const char *buf, size_t size)
        return ret;
 }
 
-static ssize_t
-console_log_read(__attribute__((unused)) void *c,
-                __attribute__((unused)) char *buf,
-                __attribute__((unused)) size_t size)
-{
-       return 0;
-}
-
-static int
-console_log_seek(__attribute__((unused)) void *c,
-                __attribute__((unused)) off64_t *offset,
-                __attribute__((unused)) int whence)
-{
-       return -1;
-}
-
-static int
-console_log_close(__attribute__((unused)) void *c)
-{
-       return 0;
-}
-
 static cookie_io_functions_t console_log_func = {
-       .read  = console_log_read,
        .write = console_log_write,
-       .seek  = console_log_seek,
-       .close = console_log_close
 };
 
 /*
@@ -150,33 +125,8 @@ early_log_write(__attribute__((unused)) void *c, const char *buf, size_t size)
        return ret;
 }
 
-static ssize_t
-early_log_read(__attribute__((unused)) void *c,
-              __attribute__((unused)) char *buf,
-              __attribute__((unused)) size_t size)
-{
-       return 0;
-}
-
-static int
-early_log_seek(__attribute__((unused)) void *c,
-              __attribute__((unused)) off64_t *offset,
-              __attribute__((unused)) int whence)
-{
-       return -1;
-}
-
-static int
-early_log_close(__attribute__((unused)) void *c)
-{
-       return 0;
-}
-
 static cookie_io_functions_t early_log_func = {
-       .read  = early_log_read,
        .write = early_log_write,
-       .seek  = early_log_seek,
-       .close = early_log_close
 };
 static FILE *early_log_stream;