* [PATCH] sparse function pointer arguments now accept void pointers
@ 2003-07-31 5:28 Dave Olien
2003-07-31 6:29 ` Bernd Eckenfels
0 siblings, 1 reply; 3+ messages in thread
From: Dave Olien @ 2003-07-31 5:28 UTC (permalink / raw)
To: torvalds; +Cc: linux-kernel
This patch eliminates warnings of the form:
incorrect type in argument 1 (different base types)
from code of the form:
#define VPTR ((void *)1)
void f( int (g)(void))
{
}
int
main(void)
{
f(VPTR);
f(0);
}
--- sparse_original/evaluate.c 2003-07-29 14:13:09.000000000 -0700
+++ sparse_patch/evaluate.c 2003-07-30 18:14:25.000000000 -0700
@@ -647,7 +653,7 @@
t = t->ctype.base_type;
target_as |= t->ctype.as;
}
- if (t->type == SYM_PTR) {
+ if (t->type == SYM_PTR || t->type == SYM_FN) {
struct expression *right = *rp;
struct symbol *s = source;
int source_as;
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] sparse function pointer arguments now accept void pointers
2003-07-31 5:28 [PATCH] sparse function pointer arguments now accept void pointers Dave Olien
@ 2003-07-31 6:29 ` Bernd Eckenfels
2003-07-31 16:45 ` Dave Olien
0 siblings, 1 reply; 3+ messages in thread
From: Bernd Eckenfels @ 2003-07-31 6:29 UTC (permalink / raw)
To: linux-kernel
In article <20030731052810.GA2853@osdl.org> you wrote:
> This patch eliminates warnings of the form:
...
> - if (t->type == SYM_PTR) {
> + if (t->type == SYM_PTR || t->type == SYM_FN) {
unlikely
Greetings
Bernd
--
eckes privat - http://www.eckes.org/
Project Freefire - http://www.freefire.org/
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] sparse function pointer arguments now accept void pointers
2003-07-31 6:29 ` Bernd Eckenfels
@ 2003-07-31 16:45 ` Dave Olien
0 siblings, 0 replies; 3+ messages in thread
From: Dave Olien @ 2003-07-31 16:45 UTC (permalink / raw)
To: Bernd Eckenfels; +Cc: linux-kernel
Bernd,
I might have messed up here. But could you give a little more explanation,
to help me understand what you think is wrong with this and why?
Dave
On Thu, Jul 31, 2003 at 08:29:24AM +0200, Bernd Eckenfels wrote:
> In article <20030731052810.GA2853@osdl.org> you wrote:
> > This patch eliminates warnings of the form:
> ...
> > - if (t->type == SYM_PTR) {
> > + if (t->type == SYM_PTR || t->type == SYM_FN) {
>
> unlikely
>
> Greetings
> Bernd
> --
> eckes privat - http://www.eckes.org/
> Project Freefire - http://www.freefire.org/
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2003-07-31 16:44 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-07-31 5:28 [PATCH] sparse function pointer arguments now accept void pointers Dave Olien
2003-07-31 6:29 ` Bernd Eckenfels
2003-07-31 16:45 ` Dave Olien
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).