[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: branches/kdepim/enterprise4/kdelibs-4.1-branch/kdeui/dialogs
From:       Thomas McGuire <mcguire () kde ! org>
Date:       2009-10-09 20:52:01
Message-ID: 200910092252.09169.mcguire () kde ! org
[Download RAW message or body]


Hi Marc,

On Wednesday 07 October 2009 11:41:47 Marc Mutz wrote:
> SVN commit 1032206 by mutz:
> 
> KAboutApplicationDialog: add options to suppress translators tab and kde
>  version.
> 
> This allows to use this dialog for other About Foo dialogs than just the
>  one for the application, as seen in the About Gpg4win dialog in Kleopatra.
> 
>  M  +35 -15    kaboutapplicationdialog.cpp
>  M  +22 -1     kaboutapplicationdialog.h
> 
> [SNIP]
>
>      }
> +    }

Can you please indent those if statements correctly? That makes the code more 
readable and less confusing.
I've seen this in 3 places now.

Also, the API documentation for the new stuff here is missing, I'll add that 
in the merge to kdelibs trunk.

Regards,
Thomas

["signature.asc" (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]