[haiku-commits] Re: r38354 - in haiku/trunk: headers/private/graphics/intel_extreme src/add-ons/accelerants/intel_extreme

  • From: Andreas Färber <andreas.faerber@xxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 25 Aug 2010 19:36:34 +0200

Am 25.08.2010 um 17:43 schrieb Adrien Destugues:


        if (intel_propose_display_mode(&target, mode, mode))
                return B_BAD_VALUE;
+               // TODO : it may be acceptable to continue when using
panel fitting,
+               // since the data from propose_display_mode will not
actually get any
+               // use


The if block should get parenthesis now, since the comment makes it a
multi-line if-block.


The TODO refers to the whole if block above, so I see it as outside of
the block.


Then it should go before the block according to my reading.
@Axel: Had it been just for the return line, it should've been indented another level, no? 

Andreas

Other related posts:

  1. Home
  2. haiku-commits
  3. Archive
  4. 08-2010