From 9ee0e8c5b5df0b114218e28753e6c0291cb356a3 Mon Sep 17 00:00:00 2001 From: solar Date: Fri, 12 May 2006 14:42:11 +0000 Subject: [PATCH] Intermediate work while migrating CVS -> SVN. --- Makefile | 2 +- functions/_PDCLIB/print.c | 11 +++++++++++ functions/stdio/fopen.c | 38 ++++++++++++++++++++++++++++---------- internals/_PDCLIB_int.h | 2 +- 4 files changed, 41 insertions(+), 12 deletions(-) diff --git a/Makefile b/Makefile index 81a353b..9aa1c89 100644 --- a/Makefile +++ b/Makefile @@ -6,7 +6,7 @@ AUXFILES := Makefile Readme.txt PROJDIRS := functions includes internals SRCFILES := $(shell find $(PROJDIRS) -mindepth 1 -maxdepth 3 -name "*.c") HDRFILES := $(shell find $(PROJDIRS) -mindepth 1 -maxdepth 3 -name "*.h") -INTFILES := atomax digits lengthmods seed strtox_main strtox_prelim rename remove _Exit +INTFILES := _Exit atomax digits lengthmods print rename remove seed strtox_main strtox_prelim OBJFILES := $(patsubst %.c,%.o,$(SRCFILES)) TSTFILES := $(patsubst %.c,%.t,$(SRCFILES)) REGFILES := $(filter-out $(patsubst %,functions/_PDCLIB/%.r,$(INTFILES)),$(patsubst %.c,%.r,$(SRCFILES))) diff --git a/functions/_PDCLIB/print.c b/functions/_PDCLIB/print.c index b0c3341..986ab43 100644 --- a/functions/_PDCLIB/print.c +++ b/functions/_PDCLIB/print.c @@ -462,3 +462,14 @@ const char * _PDCLIB_print( const char * spec, struct _PDCLIB_status_t * status } return ++spec; } + +#ifdef TEST +#include <_PDCLIB_test.h> + +int main( void ) +{ + TESTCASE( NO_TESTDRIVER ); + return TEST_RESULTS; +} + +#endif diff --git a/functions/stdio/fopen.c b/functions/stdio/fopen.c index 244549c..f110944 100644 --- a/functions/stdio/fopen.c +++ b/functions/stdio/fopen.c @@ -14,7 +14,11 @@ static FILE * _PDCLIB_filelist = NULL; -static int filemode( char const * const mode ) +/* Helper function that parses the C-style mode string passed to fopen() into + the PDCLib flags.FREAD, FWRITE, FAPPEND, FRW (read-write) and FBIN ( + mode). +*/ +static unsigned int filemode( char const * const mode ) { int rc = 0; switch ( mode[0] ) @@ -29,27 +33,31 @@ static int filemode( char const * const mode ) rc |= _PDCLIB_FAPPEND; break; default: - return -1; + /* Other than read, write, or append - invalid */ + return 0; } for ( size_t i = 1; i < 4; ++i ) { switch ( mode[1] ) { case '+': - if ( rc & _PDCLIB_FRW ) return -1; + if ( rc & _PDCLIB_FRW ) return 0; /* Duplicates are invalid */ rc |= _PDCLIB_FRW; break; case 'b': - if ( rc & _PDCLIB_FBIN ) return -1; + if ( rc & _PDCLIB_FBIN ) return 0; /* Duplicates are invalid */ rc |= _PDCLIB_FBIN; break; case '\0': + /* End of mode */ return rc; default: - return -1; + /* Other than read/write or binary - invalid. */ + return 0; } } - return -1; + /* Longer than three chars - invalid. */ + return 0; } FILE * fopen( const char * _PDCLIB_restrict filename, const char * _PDCLIB_restrict mode ) @@ -57,14 +65,24 @@ FILE * fopen( const char * _PDCLIB_restrict filename, const char * _PDCLIB_restr FILE * rc; if ( mode == NULL || filename == NULL || filename[0] == '\0' ) { + /* Mode or filename invalid */ return NULL; } - if ( ( rc = calloc( 1, sizeof( FILE ) ) ) == NULL ) return rc; /* no space for another FILE */ - if ( ( rc->status = filemode( mode ) ) == -1 ) goto fail; /* invalid mode given */ - if ( ( rc->handle = _PDCLIB_open( filename, rc->status ) ) == -1 ) goto fail; /* OS "open" failed */ + if ( ( rc = calloc( 1, sizeof( FILE ) ) ) == NULL ) + { + /* no memory for another FILE */ + return NULL; + } + if ( ( rc->status = filemode( mode ) ) == 0 ) goto fail; /* invalid mode */ + rc->handle = _PDCLIB_open( filename, rc->status ); + if ( rc->handle == _PDCLIB_NOHANDLE ) goto fail; /* OS open() failed */ + + /* Adding to list of open files */ rc->next = _PDCLIB_filelist; _PDCLIB_filelist = rc; - /* TODO: Continue here: Set up PDCLib FILE contents */ + /* Setting buffer. TODO: Check for unbuffered? */ + if ( ( rc->buffer = malloc( BUFSIZ ) ) == NULL goto fail; + /* TODO: Setting mbstate */ return rc; fail: free( rc ); diff --git a/internals/_PDCLIB_int.h b/internals/_PDCLIB_int.h index faac079..ab35e19 100644 --- a/internals/_PDCLIB_int.h +++ b/internals/_PDCLIB_int.h @@ -267,7 +267,7 @@ struct _PDCLIB_file_t _PDCLIB_fpos_t position; /* file position indicator */ void * buffer; /* file buffer */ _PDCLIB_size_t bufsize; /* size of buffer */ - int status; /* misc. status bits */ + unsigned int status; /* misc. status bits */ /*mbstate_t mbstate; multibyte parse state - TODO: Unmask. */ struct _PDCLIB_file_t * next; /* provisions for linked list handling */ }; -- 2.40.0