aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTomasz Kramkowski <tomasz@kramkow.ski>2024-05-02 21:08:11 +0100
committerTomasz Kramkowski <tomasz@kramkow.ski>2024-05-02 21:37:17 +0100
commitc4ee664d0b2f4e118106fdd49eb50d9fb3eaeee3 (patch)
treef1df32ff72151a7b2e40da3bf29db11afb89209a
parent476044b4947632ba2b03bc368a513cba55cb1103 (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.h4
-rw-r--r--src/stem.c2
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. */
diff --git a/src/stem.c b/src/stem.c
index 651d624..f623ae7 100644
--- a/src/stem.c
+++ b/src/stem.c
@@ -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);
}