Commit 77d7d891 authored by Greg Williams's avatar Greg Williams
Browse files

Force enabled printf format checking in project.yml in order to catch issues... 

Force enabled printf format checking in project.yml in order to catch issues that GCC clang is not catching with mismatched specifiers.
Ensured that FILE* is cast to size_t to eliminate platform specific complaints.
Readded sleep to teardown in socket tests, since I guess it DOES matter on Linux... UGH
parent b53d675d

project.yml

+1 −0
Original line number Diff line number Diff line
@@ -83,6 +83,7 @@ 
      - -D"$": DEFINES_TEST_PREPROCESS

      - -DGNU_COMPILER

      - -w

      - -Wformat

      - ${1}

  :test_linker:

    :executable: gcc

src/lib/kinetic_logger.c

+1 −1
Original line number Diff line number Diff line
@@ -41,7 +41,7 @@ void KineticLogger_Init(const char* logFile) 
        }

        else

        {

            fprintf(stderr, "Failed to initialize logger with file: fopen('%s') => fd=%zd", logFile, fd);

            fprintf(stderr, "Failed to initialize logger with file: fopen('%s') => fd=%zd", logFile, (size_t)fd);

            fflush(fd);

        }

    }

test/integration/test_kinetic_socket.c

+1 −0
Original line number Diff line number Diff line
@@ -69,6 +69,7 @@ void tearDown(void) 
        LOG("Shutting down socket...");

        KineticSocket_Close(FileDesc);

        FileDesc = 0;

        sleep(1);

    }

}