On 31 July 2014 09:56, Baptiste Daroussin <[email protected]> wrote:
> Author: bapt
> Date: Thu Jul 31 05:56:15 2014
> New Revision: 269323
> URL: http://svnweb.freebsd.org/changeset/base/269323
>
> Log:
> lower warning level to fix build with gcc
>
> Modified:
> head/usr.bin/m4/Makefile
>
> Modified: head/usr.bin/m4/Makefile
> ==============================================================================
> --- head/usr.bin/m4/Makefile Thu Jul 31 05:12:21 2014 (r269322)
> +++ head/usr.bin/m4/Makefile Thu Jul 31 05:56:15 2014 (r269323)
> @@ -16,6 +16,7 @@ NO_WMISSING_VARIABLE_DECLARATIONS=
> SRCS= eval.c expr.c look.c main.c misc.c gnum4.c trace.c parser.y
> tokenizer.l
> .PATH: ${.CURDIR}/lib
> SRCS+= ohash.c
> +WARNS= 3
>
> tokenizer.o: parser.h
>
>
I am suspicious of this approach.
Unless objections, I'd like to commit the following one
(it also follows the NetBSD way in fixing up .Ox m4 import).
Although it is low quality as well, it allows to keep WARNS.
Index: lib/ohash.c
===================================================================
--- lib/ohash.c (revision 269324)
+++ lib/ohash.c (working copy)
@@ -120,7 +120,7 @@
void *
ohash_remove(struct ohash *h, unsigned int i)
{
- void *result = (void *)h->t[i].p;
+ void *result = __DECONST(void *, h->t[i].p);
if (result == NULL || result == DELETED)
return NULL;
@@ -141,7 +141,7 @@
if (h->t[i].p == DELETED)
return NULL;
else
- return (void *)h->t[i].p;
+ return __DECONST(void *, h->t[i].p);
}
void *
@@ -180,7 +180,7 @@
{
for (; *pos < h->size; (*pos)++)
if (h->t[*pos].p != DELETED && h->t[*pos].p != NULL)
- return (void *)h->t[(*pos)++].p;
+ return __DECONST(void *, h->t[(*pos)++].p);
return NULL;
}
Index: misc.c
===================================================================
--- misc.c (revision 269324)
+++ misc.c (working copy)
@@ -265,7 +265,7 @@
extern char *__progname;
void
-m4errx(int eval, const char *fmt, ...)
+m4errx(int exval, const char *fmt, ...)
{
fprintf(stderr, "%s: ", __progname);
fprintf(stderr, "%s at line %lu: ", CURRENT_NAME, CURRENT_LINE);
@@ -277,7 +277,7 @@
va_end(ap);
}
fprintf(stderr, "\n");
- exit(eval);
+ exit(exval);
}
/*
--
wbr,
pluknet
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "[email protected]"
