[Kde-bindings] KDE/kdebindings/csharp/qyoto

[Richard Dale]

On Friday 01 June 2007, Arno Rehn wrote:
> SVN commit 670449 by arnorehn:
>
> * Special cased the QApplication constructor call to work correctly
>
> CCMAIL: kde-bindings at kde.org
>
>
>
>  M  +4 -0      ChangeLog
>  M  +10 -0     src/qyoto.cpp
>
>
> --- trunk/KDE/kdebindings/csharp/qyoto/ChangeLog #670448:670449
> @@ -1,3 +1,7 @@
> +2007-06-01  Arno Rehn  <arno at arnorehn.de>
> +
> +	* Special cased the QApplication constructor call to work correctly
> +
>  2007-05-31  Arno Rehn  <arno at arnorehn.de>
>
>  	* Added a hack in uics for QFrame.Shape and QFrame.Shadow
> --- trunk/KDE/kdebindings/csharp/qyoto/src/qyoto.cpp #670448:670449
> @@ -584,6 +584,16 @@
>  			ptr = _smoke->cast(_o->ptr, _o->classId, method().classId);
>  		}
>  		_items = -1;
> +
> +		/*
> +		 * special case the QApplication constructor call
> +		 * the int reference has to stay valid all the time, so create an
> additional pointer here +		 */
> +		if (isConstructor() && strcmp(_smoke->methodNames[method().name],
> "QApplication") == 0) { +			int* i = new int(_sp[1].s_int);
> +			_stack[1].s_voidp = i;
> +		}
> +
>  		(*fn)(method().method, ptr, _stack);
>
>  		if (isConstructor()) {
OK, that will work although it would be nice to think of something more 
elegant after we've got a first release done. The QCoreApplication 
constructor will need to be special cased too.

-- Richard