X-Git-Url: https://plomlompom.com/repos/?a=blobdiff_plain;f=src%2Freadwrite.c;h=09faaeb282dcb8a84731c5c1bce3c69b2dd41c77;hb=550d22ec0c3f530f5d317746f3f7e75251a1de4b;hp=e776ec2822636218908d237950ddb0939fa5bf2f;hpb=823357b90906df451402358eda7b995143922496;p=plomrogue diff --git a/src/readwrite.c b/src/readwrite.c index e776ec2..09faaeb 100644 --- a/src/readwrite.c +++ b/src/readwrite.c @@ -1,34 +1,172 @@ /* readwrite.c */ #include "readwrite.h" -#include /* for FILE typedef*/ -#include /* for uint16_t, uint32_t */ +#include /* for FILE typedef, fopen(), fgetc(), fputc(), fseek(), + * sprintf(), fwrite() + */ +#include /* for uint8_t, uint16_t, uint32_t */ +#include /* for strlen()*/ +#include /* for unlink() */ +#include "rexit.h" /* for exit_err() */ +#include "misc.h" /* for trouble_msg() */ +struct World; -/* Read/write "x" from/to "file" as bigendian representation of "size" bits. - * Only multiples of 8 allowed for "size". On failure, return 1, else 0. +/* 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.) * - * As of of now, all other read/write functions build on top of these. TODO: - * Consider externing these so-far internal functions and dropping the - * interfaces to them, instead relying on their internal validity checks. - * (Usage of padded memory copies instead of directly manipulating * x as is - * done in read_uint* would need to be added, though.) + * 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; + * } */ 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); -static uint8_t read_uintX_bigendian(FILE * file, uint32_t * x, uint8_t size) +extern FILE * try_fopen(char * path, char * mode, struct World * w, char * f) { - if (0 != size % 8) /* This bit number validity check is redundant */ - { /* as long as this function is only available */ - return 1; /* through extern interfaces pre-defining the */ - } /* bit number. TODO: Consider dropping it. */ - int16_t bitshift = size - 8; + char * msg1 = "Trouble in "; + char * msg2 = " with fopen() (mode '"; + char * msg3 = "') on path '"; + char * msg4 = "'."; + uint16_t size = strlen(msg1) + strlen(msg2) + strlen(msg3) + strlen(msg4) + + strlen(f) + strlen(path) + strlen(mode) + 1; + char msg[size]; + sprintf(msg, "%s%s%s%s%s%s%s", msg1, f, msg2, mode, msg3, path, msg4); + FILE * file_p = fopen(path, mode); + exit_err(NULL == file_p, w, msg); + return file_p; +} + + + +extern void try_fclose(FILE * file, struct World * w, char * f) +{ + char * msg = trouble_msg(w, f, "fclose()"); + exit_err(fclose(file), w, msg); + free(msg); +} + + + +extern void try_fgets(char * line, int linemax, FILE * file, + struct World * w, char * f) +{ + char * msg = trouble_msg(w, f, "fgets()"); + exit_err(NULL == fgets(line, linemax, file), w, msg); + free(msg); +} + + + +extern void try_fwrite(void * ptr, size_t size, size_t nmemb, FILE * stream, + struct World * w, char * f) +{ + char * msg = trouble_msg(w, f, "fwrite()"); + exit_err(0 == fwrite(ptr, size, nmemb, stream), w, msg); + free(msg); +} + + + +extern void try_fclose_unlink_rename(FILE * file, char * p1, char * p2, + struct World * w, char * f) +{ + try_fclose(file, w, f); + char * msg1 = "Trouble in "; + char * msg4 = "'."; + if (!access(p2, F_OK)) + { + char * msg2 = " with unlink() on path '"; + uint16_t size = strlen(msg1) + strlen(msg2) + strlen(msg4) + + strlen(f) + strlen(p2) + 1; + char msg[size]; + sprintf(msg, "%s%s%s%s%s", msg1, f, msg2, p2, msg4); + exit_err(unlink(p2), w, msg); + } + char * msg2 = " with rename() from '"; + char * msg3 = "' to '"; + uint16_t size = strlen(msg1) + strlen(f) + strlen(msg2) + strlen(p1) + + strlen(msg3) + strlen(p2) + strlen(msg4) + 1; + char msg[size]; + sprintf(msg, "%s%s%s%s%s%s%s", msg1, f, msg2, p1, msg3, p2, msg4); + exit_err(rename(p1, p2), w, msg); +} + + +extern uint16_t get_linemax(FILE * file, struct World * w, char * f) +{ + char * msg = trouble_msg(w, f, "textfile_sizes()"); + uint16_t linemax; + exit_err(textfile_sizes(file, &linemax, NULL), w, msg); + free(msg); + return linemax; +} + + + +extern uint8_t textfile_sizes(FILE * file, uint16_t * linemax_p, + uint16_t * n_lines_p) +{ + int c = 0; + uint16_t c_count = 0; + uint16_t n_lines = 0; + uint16_t linemax = 0; + while (1) + { + c = fgetc(file); + if (EOF == c) + { + break; + } + c_count++; + if ('\n' == c) + { + if (c_count > linemax) + { + linemax = c_count; + } + c_count = 0; + if (n_lines_p) + { + n_lines++; + } + } + } + if (0 == linemax && 0 < c_count) /* Handle files that consist of only one */ + { /* line / lack newline chars. */ + linemax = c_count; + } + + if (-1 == fseek(file, 0, SEEK_SET)) + { + return 1; + } + * linemax_p = linemax; + if (n_lines_p) + { + * n_lines_p = n_lines; + } + return 0; +} + + + +static uint8_t read_uintX_bigendian(FILE * file, uint32_t * x, uint8_t size) +{ * x = 0; + int16_t bitshift = size - 8; int test; for (; bitshift >= 0; bitshift = bitshift - 8) { @@ -46,12 +184,7 @@ 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) { - if (0 != size % 8) /* See comment */ - { /* on identical */ - return 1; /* code block in */ - } /* read_uintX_bigendian(). */ int16_t bitshift = size - 8; - for (; bitshift >= 0; bitshift = bitshift - 8) { if (EOF == fputc((x >> bitshift) & 0xFF, file)) @@ -79,7 +212,7 @@ 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 comment for rationale. */ + /* 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;