diff options
author | Arnold D. Robbins <arnold@skeeve.com> | 2017-10-14 21:16:39 +0300 |
---|---|---|
committer | Arnold D. Robbins <arnold@skeeve.com> | 2017-10-14 21:16:39 +0300 |
commit | 880974d61412e6d78ee014d7010486fba2e3e1be (patch) | |
tree | b52d82f1e5393c141c9f42c9d40837f6d6d95f77 /awkgram.c | |
parent | 9b85ee5535a0fd1f21b1cb43d4f5c91ef719684d (diff) | |
download | egawk-880974d61412e6d78ee014d7010486fba2e3e1be.tar.gz egawk-880974d61412e6d78ee014d7010486fba2e3e1be.tar.bz2 egawk-880974d61412e6d78ee014d7010486fba2e3e1be.zip |
Simply lint checking for functions defined/called.
Diffstat (limited to 'awkgram.c')
-rw-r--r-- | awkgram.c | 27 |
1 files changed, 12 insertions, 15 deletions
@@ -7357,22 +7357,19 @@ check_funcs() for (i = 0; i < HASHSIZE; i++) { for (fp = ftable[i]; fp != NULL; fp = fp->next) { -#ifdef REALLYMEAN - /* making this the default breaks old code. sigh. */ - if (fp->defined == 0 && ! fp->extension) { - error( - _("function `%s' called but never defined"), fp->name); - errcount++; - } -#else - if (do_lint && fp->defined == 0 && ! fp->extension) - lintwarn( - _("function `%s' called but never defined"), fp->name); -#endif + if (do_lint && ! fp->extension) { + /* + * Making this not a lint check and + * incrementing * errcount breaks old code. + * Sigh. + */ + if (fp->defined == 0) + lintwarn(_("function `%s' called but never defined"), + fp->name); - if (do_lint && fp->used == 0 && ! fp->extension) { - lintwarn(_("function `%s' defined but never called directly"), - fp->name); + if (fp->used == 0) + lintwarn(_("function `%s' defined but never called directly"), + fp->name); } } } |