From: Joerg Sonnenberger <joerg@britannica.bec.de>
To: discuss@mdocml.bsd.lv
Subject: Re: mdocml version 1.12.0 available
Date: Tue, 11 Oct 2011 16:56:41 +0200 [thread overview]
Message-ID: <20111011145641.GA25314@britannica.bec.de> (raw)
In-Reply-To: <4E90B130.3080008@bsd.lv>
[-- Attachment #1: Type: text/plain, Size: 210 bytes --]
On Sat, Oct 08, 2011 at 10:23:12PM +0200, Kristaps Dzonsons wrote:
> I'm pleased to announce mdocml 1.12.0, now at http://mdocml.bsd.lv/.
Attached is the report from clang -analyze. Food for thought ;)
Joerg
[-- Attachment #2: clang.txt --]
[-- Type: text/plain, Size: 3937 bytes --]
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/lib/libmandoc/../../dist/man_validate.c:217:10: warning: Assigned value is always the same as the existing value
for (cp = p; NULL != (p = strchr(p, '\t')); p++) {
~~ ^ ~
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/lib/libmandoc/../../dist/man_validate.c:216:2: warning: Value stored to 'cp' is never read
cp = p = n->string;
^ ~~~~~~~~~~~~~
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/lib/libmandoc/../../dist/man_validate.c:476:2: warning: Value stored to 'n' is never read
n = n->child;
^ ~~~~~~~~
3 warnings generated.
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/lib/libmandoc/../../dist/mdoc_macro.c:1449:3: warning: Value stored to 'head' is never read
head = m->last;
^ ~~~~~~~
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/lib/libmandoc/../../dist/mdoc_macro.c:1457:3: warning: Value stored to 'body' is never read
body = m->last;
^ ~~~~~~~
2 warnings generated.
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/lib/libmandoc/../../dist/mdoc_validate.c:549:10: warning: Assigned value is always the same as the existing value
for (cp = p; NULL != (p = strchr(p, '\t')); p++) {
~~ ^ ~
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/lib/libmandoc/../../dist/mdoc_validate.c:548:2: warning: Value stored to 'cp' is never read
cp = p;
^ ~
2 warnings generated.
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/lib/libmandoc/../../dist/tbl_data.c:152:29: warning: The left operand to '+' is always 0
if (p[pos] == 'T' && p[pos + 1] == '}') {
~~~ ^
1 warning generated.
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/lib/libmandoc/../../dist/mandoc.c:311:2: warning: Value stored to 'rlim' is never read
rlim = -1;
^ ~~
1 warning generated.
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/bin/mandoc/../../dist/main.c:136:2: warning: Value stored to 'argc' is never read
argc -= optind;
^ ~~~~~~
1 warning generated.
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/bin/mandoc/../../dist/mdoc_html.c:502:3: warning: Value stored to 't' is never read
t = print_otag(h, TAG_TABLE, 3, tag);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/bin/mandoc/../../dist/mdoc_html.c:507:3: warning: Value stored to 't' is never read
t = print_otag(h, TAG_TABLE, 2, tag);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 warnings generated.
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/bin/mandoc/../../dist/mdoc_term.c:617:7: warning: Access to field 'prev' results in a dereference of a null pointer (loaded from variable 'n')
if (n->prev && MDOC_It == n->prev->tok) {
^
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/bin/mandoc/../../dist/mdoc_term.c:611:7: warning: Access to field 'prev' results in a dereference of a null pointer (loaded from variable 'n')
if (n->prev && MDOC_It == n->prev->tok)
^
2 warnings generated.
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/bin/mandoc/../../dist/term.c:615:3: warning: Value stored to 'c' is never read
c = 0;
^ ~
1 warning generated.
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/bin/mandoc/../../dist/term_ps.c:492:3: warning: Value stored to 'pp' is never read
pp = "letter";
^ ~~~~~~~~
/home/joerg/work/NetBSD/cvs/src/external/bsd/mdocml/bin/mandoc/../../dist/term_ps.c:579:2: warning: Value stored to 'len' is never read
len = vsnprintf(&p->ps->psmarg[pos], PS_BUFSLOP, fmt, ap);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 warnings generated.
prev parent reply other threads:[~2011-10-11 14:57 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <sfid-H20111008-222347-+063.46-1@spamfilter.osbf.lua>
2011-10-08 20:23 ` Kristaps Dzonsons
2011-10-10 13:05 ` Thomas Klausner
2011-10-10 13:23 ` Kristaps Dzonsons
2011-10-10 14:31 ` Thomas Klausner
2011-10-10 17:14 ` Ingo Schwarze
2011-10-11 14:56 ` Joerg Sonnenberger [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20111011145641.GA25314@britannica.bec.de \
--to=joerg@britannica.bec.de \
--cc=discuss@mdocml.bsd.lv \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).