bugs in template "KDE2 Normal"

[Ralf Nolden]

Olaf Hartig wrote:

thanks, applied and I will put it into CVS later today.

Ralf
> 
> Hello,
> I've found two bugs in the template "KDE2 normal app". It's in KDevelop 1.4
> and in the latest CVS (KDEVELOP_1_4 branch) and may be in the HEAD branch
> (didn't checked it).
> They are in |Name|Doc::saveModified().
> 
> 1) The strings in the messagebox creation (KMessageBox::warningYesNoCancel)
> have to be exchanged, because we have the parameterlist
>         parent, text, caption, ...
> and not
>         parent, caption, text, ...
> 
> 2) The values for the case statements are wrong. It has to be
>         case 3:
>                 ...; break;
>         case 4:
>                 ...; break;
>         case 2:
>                 ...; break;
> Or better take the identifiers:
>         case KMessageBox::Yes:
>                 ...; break;
>         case KMessageBox::No:
>                 ...; break;
>         case KMessageBox::Cancel:
>                 ...; break;
> 
> I changed the according file ( ./skel/kbasedoc.cpp ) and attached it. Simply
> replace it in ./kdevelop/templates/normal2.tar.gz
> 
> Please CC - I'm not on the list.
> 
> Bye,
> Ole
> 
>   ------------------------------------------------------------------------
>                    Name: kbasedoc.cpp
>    kbasedoc.cpp    Type: text/x-c++
>                Encoding: 7BIT

--
We're not a company, we just produce better code at less costs.
--------------------------------------------------------------------
Ralf Nolden
nolden at kde.org

The K Desktop Environment	The KDevelop Project
http://www.kde.org		http://www.kdevelop.org



-
to unsubscribe from this list send an email to kdevelop-request at kdevelop.org with the following body:
unsubscribe »your-email-address«