On Tue, 2011-02-15 at 19:02 +0100, Axel Dörfler wrote: > kallisti5@xxxxxxxxxxx wrote: > > + */ > > > > - > > -*/ > > - > > #include "driver.h" > > Still two lines after the header, and the includes... > > > +++ haiku/trunk/src/add- > > ons/kernel/drivers/graphics/radeon_hd/radeon_hd.cpp 2011-02-14 21:25:24 > UTC (rev 40496) > > @@ -7,7 +7,6 @@ > > * Clemens Zeidler, haiku@xxxxxxxxxxxxxxxxxx > > */ > > > > - > > #include "radeon_hd.h" > > Same here, really, great commit! :-/ > > > +++ haiku/trunk/src/add- > > ons/kernel/drivers/graphics/radeon_hd/radeon_hd_private.h 2011-02-14 > 21:25:24 UTC (rev 40496) > > @@ -6,6 +6,7 @@ > > * Axel Dörfler, axeld@xxxxxxxxxxxxxxxx > > * Clemens Zeidler, haiku@xxxxxxxxxxxxxxxxxx > > */ > > + > > #ifndef RADEON_RD_PRIVATE_H > > #define RADEON_RD_PRIVATE_H > > NO BLANK LINE BETWEEN HEADER AND HEADER GUARD. > Is it really that hard? sorry, working at 1024x768 on a 1600x900 monitor is rough. I mostly threw those in because of all the Haiku Inc to Haiku, Inc header changes mmadia recommended. will fix and push on my next commit. -- Thanks! -Alexander von Gluck IV