diff options
| author | Kristaps Dzonsons <kristaps@bsd.lv> | 2011-05-26 21:13:07 +0000 |
|---|---|---|
| committer | Kristaps Dzonsons <kristaps@bsd.lv> | 2011-05-26 21:13:07 +0000 |
| commit | 7ffcd2ab33874c3e98547e16517172680e9b7217 (patch) | |
| tree | 7819499c9cf063350a2f90b4c107821a528a1290 | |
| parent | cb6de654ba5108e6591f7c7601d03a59019979f7 (diff) | |
| download | mandoc-7ffcd2ab33874c3e98547e16517172680e9b7217.tar.gz | |
Some small lint checks in preconv. Also add it to the default lint rule.
| -rw-r--r-- | Makefile | 2 | ||||
| -rw-r--r-- | preconv.c | 10 |
2 files changed, 6 insertions, 6 deletions
@@ -298,7 +298,7 @@ INDEX_OBJS = $(INDEX_MANS) \ www: index.html -lint: llib-llibmandoc.ln llib-lmandoc.ln +lint: llib-llibmandoc.ln llib-lmandoc.ln llib-lpreconv.ln clean: rm -f libmandoc.a $(LIBMANDOC_OBJS) @@ -137,7 +137,7 @@ conv_utf_8(const struct buf *b) /* Quick test for big-endian value. */ - if ( ! (*((char *)(&one)))) + if ( ! (*((const char *)(&one)))) be = 1; for (i = b->offs; i < b->sz; i++) { @@ -376,7 +376,7 @@ cue_enc(const struct buf *b, size_t *offs, enum enc *enc) /* Check us against known encodings. */ - for (i = 0; i < ENC__MAX; i++) { + for (i = 0; i < (int)ENC__MAX; i++) { nsz = strlen(encs[i].name); if (phsz < nsz) continue; @@ -403,7 +403,7 @@ main(int argc, char *argv[]) struct buf b; const char *fn; enum enc enc, def; - const char bom[3] = { 0xEF, 0xBB, 0xBF }; + unsigned char bom[3] = { 0xEF, 0xBB, 0xBF }; size_t offs; extern int optind; extern char *optarg; @@ -427,12 +427,12 @@ main(int argc, char *argv[]) case ('D'): /* FALLTHROUGH */ case ('e'): - for (i = 0; i < ENC__MAX; i++) { + for (i = 0; i < (int)ENC__MAX; i++) { if (strcasecmp(optarg, encs[i].name)) continue; break; } - if (i < ENC__MAX) { + if (i < (int)ENC__MAX) { if ('D' == ch) def = (enum enc)i; else |