Cleanup trailing whitespace
authorEKR <ekr@rtfm.com>
Mon, 27 May 2013 20:39:06 +0000 (13:39 -0700)
committerJean-Marc Valin <jmvalin@jmvalin.ca>
Mon, 27 May 2013 21:29:35 +0000 (17:29 -0400)
Conflicts:
unix/Makefile

src/logging.c
src/logging.h
src/tests/logging_test.c
unix/Makefile

index 46eab6b..1ab964e 100644 (file)
@@ -62,7 +62,7 @@ static const char *od_log_facility_name(od_log_facility fac) {
   assert (fac < OD_LOG_FACILITY_MAX);
   if (fac >= OD_LOG_FACILITY_MAX)
     return "INVALID";
-  
+
   return od_log_module_names[fac];
 }
 
@@ -72,7 +72,7 @@ static const char *od_log_level_name(od_log_level level) {
 
   if (level >= OD_LOG_LEVEL_MAX)
     return "INVALID";
-  
+
   return od_log_level_names[level];
 }
 
@@ -123,13 +123,13 @@ int od_log_init(od_logger_function logger) {
     /* At this point, we should theoretically have a valid clause */
     *colon = '\0';
     ++colon; /* Now points to the start of the number. */
-    
+
     for (i = 0; i < OD_LOG_FACILITY_MAX; ++i) {
       if (od_log_module_names[i] && !strcmp(ptr, od_log_module_names[i])) {
         break;  /* Success */
       }
     }
-    
+
     if (i == OD_LOG_FACILITY_MAX) {
       fprintf(stderr, "Unknown facility '%s'\n", ptr);
       continue;
@@ -164,7 +164,7 @@ int od_log(od_log_facility fac, od_log_level level, const char *fmt, ...) {
   va_start(ap, fmt);
   (void)od_logger(fac, level, fmt, ap);
   va_end(ap);
-  
+
   return 0;
 }
 
@@ -173,7 +173,7 @@ static int od_log_fprintf_stderr(od_log_facility facility,
                                  const char *fmt, va_list ap) {
   char fmt_buffer[1024];
   int rv;
-  
+
   rv = snprintf(fmt_buffer, sizeof(fmt_buffer),
            "[%s/%s] %s\n",
            od_log_facility_name(facility),
@@ -185,6 +185,6 @@ static int od_log_fprintf_stderr(od_log_facility facility,
   }
 
   (void)vfprintf(stderr, fmt_buffer, ap);
+
  return 0;
 }
index 642b5bc..9640d4d 100644 (file)
@@ -60,7 +60,7 @@ typedef enum {
    od_log_init() determines what levels to log at by examining
     the environment variable OD_LOG_MODULES, which is of the
     form "<facility-name>:<level>(,<facility-name>:<level>)*".
-    
+
    For instance:
     OD_LOG_MODULES=generic:5
 
index c83bd0c..985deb7 100644 (file)
@@ -47,7 +47,6 @@ static void expected_nothing() {
 }
 
 static void expected_result(const char *expected) {
-  
   if (!emitted) {
     fprintf(stderr, "ERROR: Log not called\n");
     failed = 1;
@@ -104,7 +103,7 @@ int main(int argc, char **argv) {
   reset_result();
   OD_LOG((OD_LOG_ENTROPY_CODER, OD_LOG_DEBUG, "Blah blah %s:%d", "XXX", 9));
   expected_something();
-    
+
   /* Test bogus module string */
   setenv("OD_LOG_MODULES", "generic:XXX,blahblah:9,entropy-coder:5", 1);
   od_log_init(od_logging_test_emit);
@@ -121,7 +120,7 @@ int main(int argc, char **argv) {
   bogus_fmt_string[sizeof(bogus_fmt_string) - 2] = 'Y';
   bogus_fmt_string[sizeof(bogus_fmt_string) - 1] = '\0';
   OD_LOG((OD_LOG_ENTROPY_CODER, OD_LOG_DEBUG, bogus_fmt_string, "XXX", 9));
-  
+
   if (failed)
     exit(1);
 
index 9b801e6..a9dfc69 100644 (file)
@@ -37,6 +37,8 @@ PLAYER_EXAMPLE_TARGET = player_example
 # Test targets
 ECTEST_TARGET = ectest
 TEST_COEF_CODER_TARGET = test_coef_coder
+LOGGING_TEST_TARGET = logging_test
+
 # The command to use to generate dependency information
 MAKEDEPEND = gcc -MM
 #MAKEDEPEND = makedepend -f- -Y --
@@ -59,12 +61,14 @@ CFLAGS := -std=c89 -pedantic $(CFLAGS)
 CFLAGS := -fvisibility=hidden $(CFLAGS)
 CFLAGS := -Wextra -Wno-parentheses -Wno-long-long -Wno-overlength-strings $(CFLAGS)
 CFLAGS := -Wall $(CFLAGS)
+CFLAGS := -I/opt/local/include $(CFLAGS)
 # Set machine-specific flags
 HOST := $(shell uname -m)
 ifneq (,$(findstring 86,${HOST}))
 CFLAGS := -DOD_X86ASM $(CFLAGS)
 else ifneq (,$(findstring arm,${HOST}))
 CFLAGS := -DOD_ARMASM $(CFLAGS)
+CFLAGS := -DOD_LOGGING_ENABLED $(CFLAGS)
 #CFLAGS := -march=armv7 $(CFLAGS)
 endif
 
@@ -77,6 +81,7 @@ endif
 
 ECTEST_LIBS = -lm
 TEST_COEF_CODER_LIBS = -lm
+LOGGING_TEST_LIBS = ./libdaalabase.a -lm
 
 # ANYTHING BELOW THIS LINE PROBABLY DOES NOT NEED EDITING
 CINCLUDE := -I../include ${CINCLUDE}
@@ -101,6 +106,7 @@ internal.c \
 intra.c \
 intradata.c \
 laplace_tables.c \
+logging.c \
 mc.c \
 newdct.c \
 pvq.c \
@@ -119,6 +125,7 @@ mc.h \
 entcode.h \
 entdec.h \
 entenc.h \
+logging.h \
 mfrngcod.h \
 odintrin.h \
 pvq.h \
@@ -151,6 +158,7 @@ PLAYER_EXAMPLE_CSOURCES = player_example.c
 
 ECTEST_CSOURCES = tests/ectest.c
 TEST_COEF_CODER_CSOURCES = tests/test_coef_coder.c
+LOGGING_TEST_CSOURCES=tests/logging_test.c
 
 # Create object file list.
 LIBDAALABASE_OBJS:= ${LIBDAALABASE_CSOURCES:%.c=${WORKDIR}/%.o}
@@ -167,6 +175,7 @@ ENCODER_EXAMPLE_OBJS:= ${ENCODER_EXAMPLE_CSOURCES:%.c=${WORKDIR}/%.o}
 PLAYER_EXAMPLE_OBJS:= ${PLAYER_EXAMPLE_CSOURCES:%.c=${WORKDIR}/%.o}
 ECTEST_OBJS:= ${ECTEST_CSOURCES:%.c=${WORKDIR}/%.o}
 TEST_COEF_CODER_OBJS:= ${TEST_COEF_CODER_CSOURCES:%.c=${WORKDIR}/%.o}
+LOGGING_TEST_OBJS:= ${LOGGING_TEST_CSOURCES:%.c=${WORKDIR}/%.o}
 ALL_OBJS:= ${LIBDAALABASE_OBJS} ${LIBDAALADEC_OBJS} ${LIBDAALAENC_OBJS} \
  ${DUMP_VIDEO_OBJS} ${ENCODER_EXAMPLE_OBJS} ${PLAYER_EXAMPLE_OBJS} \
  ${ECTEST_OBJS} ${TEST_COEF_CODER_OBJS}
@@ -197,10 +206,12 @@ PLAYER_EXAMPLE_TARGET:= ${TARGETBINDIR}/${PLAYER_EXAMPLE_TARGET}
 # Prepend test path to file names.
 ECTEST_TARGET:= ${TESTBINDIR}/${ECTEST_TARGET}
 TEST_COEF_CODER_TARGET:= ${TESTBINDIR}/${TEST_COEF_CODER_TARGET}
+LOGGING_TEST_TARGET:= ${TESTBINDIR}/${LOGGING_TEST_TARGET}
+
 # Complete set of targets
 ALL_TARGETS:= ${LIBDAALABASE_TARGET} ${LIBDAALADEC_TARGET} \
  ${LIBDAALAENC_TARGET} ${DUMP_VIDEO_TARGET} ${ENCODER_EXAMPLE_TARGET} \
- ${PLAYER_EXAMPLE_TARGET} ${ECTEST_TARGET} ${TEST_COEF_CODER_TARGET}
+ ${PLAYER_EXAMPLE_TARGET} ${ECTEST_TARGET} ${TEST_COEF_CODER_TARGET} ${LOGGING_TEST_TARGET}
 
 # Targets:
 # Everything (default)
@@ -209,17 +220,17 @@ all: ${ALL_TARGETS}
 # libdaalabase
 ${LIBDAALABASE_TARGET}: ${LIBDAALABASE_OBJS}
        mkdir -p ${TARGETLIBDIR}
-       ar cqs $@ ${LIBDAALABASE_OBJS}
+       ar crs $@ ${LIBDAALABASE_OBJS}
 
 # libdaaladec
 ${LIBDAALADEC_TARGET}: ${LIBDAALADEC_OBJS}
        mkdir -p ${TARGETLIBDIR}
-       ar cqs $@ ${LIBDAALADEC_OBJS}
+       ar crs $@ ${LIBDAALADEC_OBJS}
 
 # libdaalaenc
 ${LIBDAALAENC_TARGET}: ${LIBDAALAENC_OBJS}
        mkdir -p ${TARGETLIBDIR}
-       ar cqs $@ ${LIBDAALAENC_OBJS}
+       ar crs $@ ${LIBDAALAENC_OBJS}
 
 # dump_video
 ${DUMP_VIDEO_TARGET}: ${DUMP_VIDEO_OBJS} ${LIBDAALABASE_TARGET} \
@@ -252,6 +263,11 @@ ${TEST_COEF_CODER_TARGET}: ${TEST_COEF_CODER_OBJS}
        mkdir -p ${TESTBINDIR}
        ${CC} ${CFLAGS} ${TEST_COEF_CODER_OBJS} ${TEST_COEF_CODER_LIBS} -o $@
 
+# logging_test
+${LOGGING_TEST_TARGET}: ${LOGGING_TEST_OBJS} ${LIBDAALABASE_TARGET}
+       mkdir -p ${TESTBINDIR}
+       ${CC} ${CFLAGS} ${LOGGING_TEST_OBJS} ${LOGGING_TEST_LIBS} -o $@
+
 # Assembly listing
 ALL_ASM := ${ALL_OBJS:%.o=%.s}
 asm: ${ALL_ASM}
@@ -260,6 +276,7 @@ asm: ${ALL_ASM}
 check: all
        ${ECTEST_TARGET}
        ${TEST_COEF_CODER_TARGET}
+       ${LOGGING_TEST_TARGET}
 
 # Remove all targets.
 clean: