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

List:       kde-commits
Subject:    Re: kdemultimedia/noatun/modules/metatag
From:       Neil Stevens <neil () qualityassistant ! com>
Date:       2003-05-11 12:29:06
[Download RAW message or body]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Sunday May 11, 2003 04:53, Daniel Molkentin wrote:
> On Sunday 11 May 2003 11:29, Neil Stevens wrote:
> >On Saturday May 10, 2003 01:42, Daniel Molkentin wrote:
> >> On Saturday 10 May 2003 22:29, Charles Samuels wrote:
> >> >Why is this necessary?
> >>
> >> It yields an amigous overload on the ctor (int, int, int..., vs.
> >> QWidget, int, int) on gcc 3.3pre.
> >
> >You can't expect to "fix" every app in existance to work with a
> > pre-release compiler this way.  Even if you "fix" every app in CVS,
> > everything outside will still fail.
>
> Sorry, this version comes with SuSE 8.2.

So?

> >Charles is right.  Fix it in a library.
>
> And am I supposed to fix that without breaking BC? Adding an empty
> parameter?

Hmm, I just looked, and I don't see a KDialogBase constructor that starts 
with int, int, int.  So which two are ambiguous?

KDialogBase( QWidget *parent=0, const char *name=0, bool modal=true,
KDialogBase( int dialogFace, const QString &caption, int buttonMask,
KDialogBase( int dialogFace, const QString &caption, QWidget *parent=0
KDialogBase( const QString &caption, int buttonMask, ButtonCode 
defaultButton

With the third one having been added recently (kdialogbase.h revision 
1.83).  Perhaps that new constructor should be removed?

- -- 
Neil Stevens - neil@qualityassistant.com
"The shepherd drives the wolf from the sheep's throat, for which the
sheep thanks the shepherd as a liberator, while the wolf denounces him
for the same act as the destroyer of liberty." -- Abraham Lincoln
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQE+vkISf7mnligQOmERAoZWAJ43lpjPi5/U+rjzgLD4UNUbk2DUJwCgmKN1
LOGqp4NjC3Fi3W3TTaoichA=
=btHV
-----END PGP SIGNATURE-----


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

Configure | About | News | Add a list | Sponsored by KoreLogic