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 + // useThe 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