From 76bcc7383e9b111e02e83c2d822633cf1be88aab Mon Sep 17 00:00:00 2001 From: Christian Heller Date: Tue, 26 Nov 2013 03:30:55 +0100 Subject: [PATCH] Removed unused code in readwrite library. --- src/readwrite.c | 32 ++------------------------------ src/readwrite.h | 2 -- 2 files changed, 2 insertions(+), 32 deletions(-) diff --git a/src/readwrite.c b/src/readwrite.c index e5db306..bfbd4f6 100644 --- a/src/readwrite.c +++ b/src/readwrite.c @@ -14,18 +14,8 @@ /* Read/write "x" from/to "file" as bigendian representation of "size" bits. On * failure, return 1, else 0. (As of of now, all extern read/write functions - * build on top of these.) - * - * Only use multiples of 8 greater or equal 32 for "size", so that storage - * inside uint32_t is possible. Originally a bit number check prefaced the code - * of both functions. It was removed as redundant due to all possible "size" - * values being hardcoded into the library (i.e. in all extern functions calling - * / wrapping around either function). If this ever changes, (re-)insert: - * - * if (0 == size || size > 32 || 0 != size % 8) - * { - * return 1; - * } + * build on top of these.) Only use multiples of 8 greater or equal 32 for + * "size", so that storage inside uint32_t is possible. */ static uint8_t read_uintX_bigendian(FILE * file, uint32_t * x, uint8_t size); static uint8_t write_uintX_bigendian(FILE * file, uint32_t x, uint8_t size); @@ -202,17 +192,6 @@ extern uint8_t read_uint8(FILE * file, uint8_t * x) -extern uint8_t read_uint16_bigendian(FILE * file, uint16_t * x) -{ - /* See read_uint8() introductory code comment for rationale. */ - uint32_t y = * x; - uint8_t err = read_uintX_bigendian(file, &y, 16); - * x = (uint16_t) y; - return err; -} - - - extern uint8_t read_uint32_bigendian(FILE * file, uint32_t * x) { return read_uintX_bigendian(file, x, 32); @@ -227,13 +206,6 @@ extern uint8_t write_uint8(uint8_t x, FILE * file) -extern uint8_t write_uint16_bigendian(uint16_t x, FILE * file) -{ - return write_uintX_bigendian(file, x, 16); -} - - - extern uint8_t write_uint32_bigendian(uint32_t x, FILE * file) { return write_uintX_bigendian(file, x, 32); diff --git a/src/readwrite.h b/src/readwrite.h index 6f7bce3..58bf4f2 100644 --- a/src/readwrite.h +++ b/src/readwrite.h @@ -53,10 +53,8 @@ extern uint8_t textfile_sizes(FILE * file, uint16_t * linemax_p, * fgetc()/fputc() failure. */ extern uint8_t read_uint8(FILE * file, uint8_t * x); -extern uint8_t read_uint16_bigendian(FILE * file, uint16_t * x); extern uint8_t read_uint32_bigendian(FILE * file, uint32_t * x); extern uint8_t write_uint8(uint8_t x, FILE * file); -extern uint8_t write_uint16_bigendian(uint16_t x, FILE * file); extern uint8_t write_uint32_bigendian(uint32_t x, FILE * file); -- 2.30.2