RE: [PATCH] qed: add missing header dependencies
From: Yuval Mintz
Date: Wed Sep 07 2016 - 08:46:20 EST
> > While I obviously have no strong objection for including
> > qed_selftest.h from qed_selftest.c, I'm not sure I understand which C
> > standard dictates this requirement.
> > Why should a function definition [not call] be preceded by a prototype?
>
> - When a function is defined in one file and used in another, you want
> both files to include the same header that has the declaration to
> ensure that the types are identical. There are cases where the
> prototype is changed after the fact in an incompatible way, causing
> silent data corruption on some configurations but maybe not on others.
O.k., motivation is clear.
But this really isn't enforced by the ansi-c standard, right?
Anyway, thanks.
Acked-by: Yuval Mintz <Yuval.Mintz@xxxxxxxxxx>