Dave Peterson <[email protected]> wrote:
>
> This is part 1 of a 2-part patch set. The code changes are split into
> two parts to make the patches more readable.
Will the code compile and run with just #1-of-2 applied?
If not, we should combine the patches (which I can do in a jiffy). Because
hitting a won't-compile in the middle of a git-bisect session is quite
painful.
Similarly we should aim for compiles-and-works at each step of the whole
series, if possible/sane.
> Move complete_mc_list_del() and del_mc_from_global_list() so we can
> call del_mc_from_global_list() from edac_mc_add_mc() without forward
> declarations. Perhaps using forward declarations would be better?
> I'm doing things this way because the rest of the code is missing
> them.
Well I prefer it the way you've done it in this patch. But my first
language was Pascal ;) (yes, they had computers then)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
