Re: [Development] Qt 5.2 header diff: QtWidgets

2013-11-07 Thread Sorvig Morten
On 05 Nov 2013, at 19:17, Shaw Andy wrote: > IIRC it wasn’t even compiled into the QtWidgets library, although the > documentation and everything existed, those symbols were never in Qt. Morten > can say 100% at least but that is my understanding and recollection at least. That is correct, th

Re: [Development] Qt 5.2 header diff: QtWidgets

2013-11-05 Thread Shaw Andy
t.org [mailto:development-bounces+andy.shaw=digia@qt-project.org] On Behalf Of Jake Petroules Sent: 5. november 2013 19:10 To: Thiago Macieira Cc: development@qt-project.org Subject: Re: [Development] Qt 5.2 header diff: QtWidgets On Nov 5, 2013, at 12:55 PM, Thiago Macieira mailto:thiago.

Re: [Development] Qt 5.2 header diff: QtWidgets

2013-11-05 Thread Jake Petroules
On Nov 5, 2013, at 12:55 PM, Thiago Macieira wrote: > On terça-feira, 5 de novembro de 2013 12:39:55, Jake Petroules wrote: >> On Nov 5, 2013, at 12:22 PM, Marc Mutz wrote: >>> But can be overloaded on the new types, right. >> >> Does this 100% NEED to be reverted? It wasn't even functional unt

Re: [Development] Qt 5.2 header diff: QtWidgets

2013-11-05 Thread Thiago Macieira
On terça-feira, 5 de novembro de 2013 12:39:55, Jake Petroules wrote: > On Nov 5, 2013, at 12:22 PM, Marc Mutz wrote: > > But can be overloaded on the new types, right. > > Does this 100% NEED to be reverted? It wasn't even functional until 5.2 > IIRC. > > I really hope this does not have to wai

Re: [Development] Qt 5.2 header diff: QtWidgets

2013-11-05 Thread Jake Petroules
On Nov 5, 2013, at 12:22 PM, Marc Mutz wrote: > On Tuesday, November 05, 2013 17:51:25 Thiago Macieira wrote: >> On terça-feira, 5 de novembro de 2013 11:09:48, Giuseppe D'Angelo wrote: class Q_WIDGETS_EXPORT QMacCocoaViewContainer : public QWidget { Q_OBJECT >>>

Re: [Development] Qt 5.2 header diff: QtWidgets

2013-11-05 Thread Marc Mutz
On Tuesday, November 05, 2013 17:51:25 Thiago Macieira wrote: > On terça-feira, 5 de novembro de 2013 11:09:48, Giuseppe D'Angelo wrote: > > > class Q_WIDGETS_EXPORT QMacCocoaViewContainer : public QWidget > > > { > > > > > > Q_OBJECT > > > > > > public: > > > -QMacCocoaViewCo

Re: [Development] Qt 5.2 header diff: QtWidgets

2013-11-05 Thread Thiago Macieira
On terça-feira, 5 de novembro de 2013 11:09:48, Giuseppe D'Angelo wrote: > > +#ifdef Q_OS_OSX > > +NSMenu* toNSMenu(); > > +void setAsDockMenu(); > > +#endif > > Is it OK to reintroduce platform specific calls in the APIs? I would've > expected this kind of functionality to be provided by

Re: [Development] Qt 5.2 header diff: QtWidgets

2013-11-05 Thread Thiago Macieira
On terça-feira, 5 de novembro de 2013 11:09:48, Giuseppe D'Angelo wrote: > > class Q_WIDGETS_EXPORT QMacCocoaViewContainer : public QWidget > > { > > Q_OBJECT > > > > public: > > -QMacCocoaViewContainer(void *cocoaViewToWrap, QWidget *parent = 0); > > +QMacCocoaViewContainer(NSV

Re: [Development] Qt 5.2 header diff: QtWidgets

2013-11-05 Thread Giuseppe D'Angelo
Il 05/11/2013 01:07, Thiago Macieira ha scritto: Pastebin URL (expires in 24 hours): http://paste.fedoraproject.org/51643/61145013 --- diff --git a/src/widgets/dialogs/qcolordialog.h b/src/widgets/dialogs/qcolordialog.h index 80a31c4..c7a1d6f 100644 --- a/src/widgets/dialogs/qcolordialog.h +++

[Development] Qt 5.2 header diff: QtWidgets

2013-11-04 Thread Thiago Macieira
Pastebin URL (expires in 24 hours): http://paste.fedoraproject.org/51643/61145013 --- diff --git a/src/widgets/dialogs/qcolordialog.h b/src/widgets/dialogs/qcolordialog.h index 80a31c4..c7a1d6f 100644 --- a/src/widgets/dialogs/qcolordialog.h +++ b/src/widgets/dialogs/qcolordialog.h @@ -112,6 +112