Re: [2.6 patch] move EXPORT_SYMBOL's away from sound/pci/emu10k1/emu10k1_main.c

[Takashi Iwai <tiwai@suse.de>]

At Fri, 7 Apr 2006 20:49:09 +0200,
Sam Ravnborg wrote:
> 
> On Fri, Apr 07, 2006 at 02:34:56PM +0200, Takashi Iwai wrote:
> > At Fri, 7 Apr 2006 02:31:05 +0200,
> > Adrian Bunk wrote:
> > > 
> > > This patch moves the EXPORT_SYMBOL's from 
> > > sound/pci/emu10k1/emu10k1_main.c to the files with the actual functions.
> > 
> > What is the merit of this movement?
> 
> 1) Documentation - it is obvious that the function/data is exported so
> be a bit mroe careful when introducing changes
> 2) Style. In 2.6 the preferred style is to put the EXPORT_SYMBOL on the
> line following the closing } of the exported function.
> 3) Keep changes local. If one removes a previously exported symbol less
> files needs to be touched.

I know the above for the new codes, yes.  But my question is wheter do
we get a good enough benifit by changing the existing code.

I'm not against such an action but just wornder whether it's really
needed.  If yes, we should do it over the whole tree.


Takashi
-
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/