diff options
author | Michał Łyszczek <michal.lyszczek@bofc.pl> | 2018-10-10 18:03:23 +0200 |
---|---|---|
committer | Michał Łyszczek <michal.lyszczek@bofc.pl> | 2018-10-10 18:03:23 +0200 |
commit | 00b3dca5cfa19b314471e877a432cdabb9500935 (patch) | |
tree | d14aab3346b025641961a385900132a2dc87afa1 | |
parent | 3e3914564ed0ef1eed1c25c98b629626952abc72 (diff) | |
download | u3-00b3dca5cfa19b314471e877a432cdabb9500935.tar.gz u3-00b3dca5cfa19b314471e877a432cdabb9500935.tar.bz2 u3-00b3dca5cfa19b314471e877a432cdabb9500935.zip |
seq: dont perform checks that considered invalid by c standard
-rw-r--r-- | src/seq.c | 13 | ||||
-rw-r--r-- | tst/seq-test.c | 16 |
2 files changed, 4 insertions, 25 deletions
@@ -73,17 +73,6 @@ static int get_number /*~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~*/ - if (num == NULL) - { - /* this shouldn't happen as this situation is forbidden by - * the C standard, but we all know things just happen - */ - - fprintf(stderr, "e/number string is NULL\n"); - errno = EINVAL; - return -1; - } - if (*num == '\0') { fprintf(stderr, "e/number is an empty string\n"); @@ -140,7 +129,7 @@ int u3_seq_main if (argc == 2) { - if (argv[1] && argv[1][0] == '-') + if (argv[1][0] == '-') { if (argv[1][1] == 'v') { diff --git a/tst/seq-test.c b/tst/seq-test.c index 8a5498b..4b44898 100644 --- a/tst/seq-test.c +++ b/tst/seq-test.c @@ -494,7 +494,6 @@ static void invalid_tests(void) { "", "", "", EINVAL }, { "", "", "123f", EINVAL }, { "", "", "f123", EINVAL }, - { "", "", NULL, EINVAL }, { "", "", "LONG_MAX", ERANGE }, { "", "", "LONG_MIN", ERANGE }, @@ -508,13 +507,11 @@ static void invalid_tests(void) { "2", "", "123f", EINVAL }, { "3", "", "f123", EINVAL }, - { "4", "", NULL, EINVAL }, { "5", "", "LONG_MAX", ERANGE }, { "6", "", "LONG_MIN", ERANGE }, { "123f", "", "2", EINVAL }, { "f123", "", "3", EINVAL }, - { NULL, "", "4", EINVAL }, { "LONG_MAX", "", "5", ERANGE }, { "LONG_MIN", "", "6", ERANGE }, @@ -528,50 +525,43 @@ static void invalid_tests(void) { "2", "2", "123f", EINVAL }, { "3", "3", "f123", EINVAL }, - { "4", "4", NULL, EINVAL }, { "5", "5", "LONG_MAX", ERANGE }, { "6", "6", "LONG_MIN", ERANGE }, { "2", "123f", "2", EINVAL }, { "3", "f123", "3", EINVAL }, - { "4", NULL, "4", EINVAL }, { "5", "LONG_MAX", "5", ERANGE }, { "6", "LONG_MIN", "6", ERANGE }, { "1", "0", "7", EINVAL }, { "2", "123f", "123f", EINVAL }, { "3", "f123", "f123", EINVAL }, - { "4", NULL, NULL, EINVAL }, { "5", "LONG_MAX", "LONG_MAX", ERANGE }, { "6", "LONG_MIN", "LONG_MIN", ERANGE }, - { "7", 0, "LONG_MIN", ERANGE }, + { "7", "0", "LONG_MIN", ERANGE }, { "123f", "2", "2", EINVAL }, { "f123", "3", "3", EINVAL }, - { NULL, "4", "4", EINVAL }, { "LONG_MAX", "5", "5", ERANGE }, { "LONG_MIN", "6", "6", ERANGE }, { "123f", "2", "123f", EINVAL }, { "f123", "3", "f123", EINVAL }, - { NULL, "4", NULL, EINVAL }, { "LONG_MAX", "5", "LONG_MAX", ERANGE }, { "LONG_MIN", "6", "LONG_MIN", ERANGE }, { "123f", "123f", "2", EINVAL }, { "f123", "f123", "3", EINVAL }, - { NULL, NULL, "4", EINVAL }, { "LONG_MAX", "LONG_MAX", "5", ERANGE }, { "LONG_MIN", "LONG_MIN", "6", ERANGE }, - { "LONG_MIN", 0, "7", ERANGE }, + { "LONG_MIN", "0", "7", ERANGE }, { "", "", "", EINVAL }, { "123f", "123f", "123f", EINVAL }, { "f123", "f123", "f123", EINVAL }, - { NULL, NULL, NULL, EINVAL }, { "LONG_MAX", "LONG_MAX", "LONG_MAX", ERANGE }, { "LONG_MIN", "LONG_MIN", "LONG_MIN", ERANGE }, - { "LONG_MIN", 0, "LONG_MIN", ERANGE }, + { "LONG_MIN", "0", "LONG_MIN", ERANGE }, { NULL, NULL, NULL, 0 } }; |