[PATCH] Remove unnecessary #includes

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

 



2011/6/22 Colin Guthrie <gmane at colin.guthr.ie>:
> 'Twas brillig, and Maarten Bosmans at 22/06/11 15:56 did gyre and gimble:
>> These are all unnecessary.
>>
>> ?227 files changed, 76 insertions(+), 472 deletions(-)
>
> /me wonders what the compile time difference is.... :)

I wondered about that too, but it turned out to be disappointingly
little (2s or so).

> I did wonder how many of the includes are unneeded recently after doing
> a bit of copy+paste hacking :) Nice to see you do the actual hard work!

yup, the copy-pasting was very visible at some places. Of course I'm
not claiming that every include is where it should be now, but the
current patch is already way past sensible from a diminishing returns
point of view.

> I'll do a distcheck etc. and then push it out if all is well. I guess it
> is fine on win32 too? Maybe Daniel can check on OSX also....?

yup win32 is fine. And from now on I'll do my pre-submit distcheck
with things like fftw and jack enabled too, that stuff gets sanity
checked also before I send it to the list.

Maarten

> Cheers
>
> Col


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux