diff options
author | Tomasz Kramkowski <tomasz@kramkow.ski> | 2024-05-02 21:08:11 +0100 |
---|---|---|
committer | Tomasz Kramkowski <tomasz@kramkow.ski> | 2024-05-02 21:37:17 +0100 |
commit | c4ee664d0b2f4e118106fdd49eb50d9fb3eaeee3 (patch) | |
tree | f1df32ff72151a7b2e40da3bf29db11afb89209a | |
parent | 476044b4947632ba2b03bc368a513cba55cb1103 (diff) |
Don't check parser_t.c against EOF
parser_t.c is a char which may be unsigned on some platforms and
therefore incapable of holding EOF.
This field is only ever set from parser_t.source which is a char array,
which only ever originates from {main,strquote,include} -> parser_pp ->
{init_parser,parser_reset} which all assign it from char arrays.
This means it should never be EOF, so the check can be dropped.
Also update the documentation so it matches.
-rw-r--r-- | include/stem.h | 4 | ||||
-rw-r--r-- | src/stem.c | 2 |
2 files changed, 2 insertions, 4 deletions
diff --git a/include/stem.h b/include/stem.h index 06f7351..c467286 100644 --- a/include/stem.h +++ b/include/stem.h @@ -48,7 +48,7 @@ struct VALUE_STRUCT { /*! @brief Parser implementation directly parses without lexer */ /*! the parser data structure parses a string of valid stem code and - * returns a value until it reaches EOF or end of string. */ + * returns a value until it reaches end of string. */ typedef struct PARSER_STRUCT { /*! @brief The string that contains valid stem code. */ char *source; @@ -181,7 +181,7 @@ value_t *parse_word(parser_t *p); /*! Error in parsing strings can occur if wrong escape code. */ void parser_error(parser_t *p); -/*! Gets the next value_t from the string, returns NULL if EOF. */ +/*! Gets the next value_t from the string, returns NULL if end of string. */ value_t *parser_get_next(parser_t *p); /*! Allocates memory for new node struct. */ @@ -331,8 +331,6 @@ value_t *parser_get_next(parser_t *p) { return parse_quote(p); case '\0': return NULL; - case EOF: - return NULL; default: return parse_word(p); } |