Skip to content

Commit

Permalink
Fixing pending problems
Browse files Browse the repository at this point in the history
The consume.c module was still failing to compile because the definitions from
common.h were not being included into it. Now they are, and in the process I
added an include guard in common.h in case it gets included in the future more
than once.

Signed-off-by: Rodrigo Tobar <[email protected]>
  • Loading branch information
rtobar committed Jun 21, 2018
1 parent 17f7fda commit a34d7ae
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
7 changes: 6 additions & 1 deletion common.h
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,13 @@
*
*/

#ifndef _COMMON_H_
#define _COMMON_H_

#if defined(WIN32) || defined(_WIN32) || defined(_MSC_VER) || defined(__MINGW32__)
#define _IS_WINDOWS
#else
#undef _IS_WINDOWS
#endif
#endif

#endif // _COMMON_H_
5 changes: 4 additions & 1 deletion consume.c
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@
*
*/

#include "common.h"

#ifndef _IS_WINDOWS

#include <errno.h>
#include <fcntl.h>
#include <math.h>
Expand All @@ -34,7 +38,6 @@

#include "crc32c.h"

#ifndef _IS_WINDOWS
int _crc32c_read_crc_write(int fd_in, int fd_out,
float in_timeout,
size_t buffsize, uint64_t total,
Expand Down

0 comments on commit a34d7ae

Please sign in to comment.