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

List:       koffice-devel
Subject:    Re: Review Request: Fix embedding kpresenter in konqueror
From:       Thomas Zander <zander () kde ! org>
Date:       2009-02-27 13:22:35
Message-ID: 200902271422.35676.zander () kde ! org
[Download RAW message or body]

On Wednesday 25. February 2009 05:09:10 Thorsten Zachmann wrote:
> Looks good. There is only one style issue I like to get fixed before you
> commit. Can you please always use {} even if there is only one line in the
> if e.g.
>
> if (shell) {
>     m_documentStructureDocker->setMasterMode(master);
> }

Notice that the coding style doesn't make this mandatory.

-- 
Thomas Zander
_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel

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