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

List:       koffice-devel
Subject:    Re: Font dialog
From:       Clarence Dang <dang () kde ! org>
Date:       2005-06-21 7:58:29
Message-ID: 200506211803.09092.dang () kde ! org
[Download RAW message or body]

On Fri, 17 Jun 2005 11:28 pm, David Faure wrote:
> On Monday 06 June 2005 23:06, pierre.stirnweiss_koffice@gadz.org wrote:
> int koLayoutTab::getOffsetFromBaseline()
> {
>         switch ( positionButtonGroup->selectedId() )
>         {
>         case 0:
>                 return 0;
>                 break;
> No need for break; after a return. Return already stops there and returns
> immediately. (You can remove many "breaks" in that file).
>

Different school of thought I guess: :)

kdegraphics/kolourpaint/kpviewscrollablecontainer.cpp:

const QCursor &kpGrip::cursorForType (GripType type)
{
    switch (type)
    {
    case Bottom:
        return Qt::sizeVerCursor;
        break;  // one day you'll forget

i.e. in case the "return" changes to something else and you forget to add 
"break".

_______________________________________________
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]