[haiku-commits] Re: r40496 - in haiku/trunk/src: add-ons/kernel/drivers/graphics/radeon_hd tests/kits/net/preflet/InterfacesAddOn

  • From: Alexander von Gluck <kallisti5@xxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 15 Feb 2011 12:16:29 -0600

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


Other related posts: