[haiku-commits] Re: r37305 - haiku/trunk/src/preferences/screensaver

  • From: "Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 30 Jun 2010 15:50:54 +0200

korli@xxxxxxxxxxxxxxxx wrote:
> +                     if (be_roster->Launch(SCREEN_BLANKER_SIG, &fSettings.
> Message(),
> +                             &fScreenSaverTestTeamId) == B_OK)
> +                             break;

This introduces a number of coding style violations that I fixed in 
r37313 - among a lot more, so this is just for reference:
* wrong indentation of the second line.

> +                     BEntry entry(path.Path());
> +                     entry_ref ref;
> +                     if (entry.GetRef(&ref) == B_OK)
> +                             be_roster->Launch(&ref, &fSettings.Message(),
> +                                     &fScreenSaverTestTeamId);

* Multiline statement following if() is not enclosed in {}.

Bye,
   Axel.


Other related posts: