[haiku-development] Re: [patch] Add resolution listing to "screenmode"
- From: "Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx>
- To: haiku-development@xxxxxxxxxxxxx
- Date: Sun, 08 Mar 2009 23:15:33 +0100 CET
PulkoMandy <pulkomandy@xxxxxxxxx> wrote:
> This patches src/bin/screenmode/screenmode.cpp and adds option -l
> (--list), which lists all available screenmodes.
Thanks for the patch, it looks good for the most part, but there are a
few issues with the coding style:
> + for(modeNumber=0;modeNumber<kmodeCount;modeNumber++
> ){
> + currentMode = screenMode.ModeAt(modeNumber);
First of all, the new block uses spaces for indenting, and then there
are a number of spaces to be inserted in the first of the above lines.
Bye,
Axel.
Other related posts:
