Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/dev



On Fri, May 27, 2011 at 7:54 AM, Christos Zoulas 
<christos%astron.com@localhost> wrote:
> In article <20110526071257.GA16138%netbsd.org@localhost>,
> David Holland  <dholland-sourcechanges%netbsd.org@localhost> wrote:
>>On Wed, May 25, 2011 at 04:33:38PM +0000, Masao Uebayashi wrote:
>> > Modified Files:
>> >     src/sys/dev/bluetooth: bcsp.c bthub.c btuart.c
>> >     src/sys/dev/ieee1394: fwdev.c fwmem.c fwohci.c
>> >
>> > Log Message:
>> > Declare cfdrivers using extern rather than including ioconf.h.
>>
>>This is wrong. Please revert it.
>>
>>The purpose of declaring things in header files is to make sure all
>>uses are consistent.
>
> Agreed, what are you trying to achieve here?

Modularity.


Home | Main Index | Thread Index | Old Index