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