From: Etienne Millon <me@emillon.org>
Date: Tue, 31 Mar 2015 08:48:54 +0200
Subject: Check return values

  - Make efread/efwrite check their return values.
    This can be done with an assert() since the actual return value is not used.

  - Patch a call to fread.

Last-Update: 2015-01-05
Forwarded: https://bitbucket.org/opentyrian/opentyrian/pull-request/3
---
 src/config.c |  3 ++-
 src/file.c   | 21 ++++++++++++++-------
 src/file.h   |  9 ++-------
 3 files changed, 18 insertions(+), 15 deletions(-)

diff --git a/src/config.c b/src/config.c
index e22622f..56b5123 100644
--- a/src/config.c
+++ b/src/config.c
@@ -223,7 +223,8 @@ cJSON *load_json( const char *filename )
 	size_t buffer_len = ftell_eof(f);
 	char *buffer = malloc(buffer_len + 1);
 	
-	fread(buffer, 1, buffer_len, f);
+	size_t res = fread(buffer, 1, buffer_len, f);
+	assert(res == buffer_len);
 	buffer[buffer_len] = '\0';
 	
 	fclose(f);
diff --git a/src/file.c b/src/file.c
index 61d612c..5cbe3ce 100644
--- a/src/file.c
+++ b/src/file.c
@@ -21,6 +21,7 @@
 
 #include "SDL.h"
 #include <errno.h>
+#include <assert.h>
 
 const char *custom_data_dir = ".";
 
@@ -123,12 +124,12 @@ long ftell_eof( FILE *f )
 	return size;
 }
 
-#if SDL_BYTEORDER == SDL_BIG_ENDIAN
 // endian-swapping fread
-size_t efread( void *buffer, size_t size, size_t num, FILE *stream )
+void efread( void *buffer, size_t size, size_t num, FILE *stream )
 {
 	size_t f = fread(buffer, size, num, stream);
 	
+#if SDL_BYTEORDER == SDL_BIG_ENDIAN
 	switch (size)
 	{
 		case 2:
@@ -146,15 +147,17 @@ size_t efread( void *buffer, size_t size, size_t num, FILE *stream )
 		default:
 			break;
 	}
-	
-	return f;
+#endif
+
+	assert(f == num);
 }
 
 // endian-swapping fwrite
-size_t efwrite( void *buffer, size_t size, size_t num, FILE *stream )
+void efwrite( void *buffer, size_t size, size_t num, FILE *stream )
 {
 	void *swap_buffer;
 	
+#if SDL_BYTEORDER == SDL_BIG_ENDIAN
 	switch (size)
 	{
 		case 2:
@@ -176,12 +179,16 @@ size_t efwrite( void *buffer, size_t size, size_t num, FILE *stream )
 			swap_buffer = buffer;
 			break;
 	}
+#else
+	swap_buffer = buffer;
+#endif
 	
 	size_t f = fwrite(swap_buffer, size, num, stream);
 	
+#if SDL_BYTEORDER == SDL_BIG_ENDIAN
 	if (swap_buffer != buffer)
 		free(swap_buffer);
+#endif
 	
-	return f;
+	assert(f == num);
 }
-#endif
diff --git a/src/file.h b/src/file.h
index 735a73d..8345abd 100644
--- a/src/file.h
+++ b/src/file.h
@@ -40,13 +40,8 @@ bool dir_file_exists( const char *dir, const char *file );
 long ftell_eof( FILE *f );
 
 // endian-swapping fread/fwrite
-#if SDL_BYTEORDER == SDL_BIG_ENDIAN
-size_t efread( void *buffer, size_t size, size_t num, FILE *stream );
-size_t efwrite( void *buffer, size_t size, size_t num, FILE *stream );
-#else
-#define efread fread
-#define efwrite fwrite
-#endif
+void efread( void *buffer, size_t size, size_t num, FILE *stream );
+void efwrite( void *buffer, size_t size, size_t num, FILE *stream );
 
 #endif // FILE_H