On Thu, 17 May 2007, David Woodhouse wrote:
> -- it found a few bugs with mismatching prototypes, such as the fact
> that ipxrtr_route_packet() actually takes a 'len' argument of type
> 'size_t', but is declared and called in af_ipx.c as if that argument is
> 'int'. (Oops, I thought I'd chased all of combine-diff-1-fixes.patch
> upstream already).

I guess af_ipx.c didn't include the relevant header file?

This is something sparse can find out now, too (as I found out
yesterday): if something is non-static, sparse gives a warning if it hasn't
seen a prototype in a header file.

Gr{oetje,eeting}s,

                                                Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [EMAIL PROTECTED]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                                            -- Linus Torvalds
-
To unsubscribe from this list: send the line "unsubscribe linux-arch" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to