Trolltech Home | QSA-interest Home | Recent Threads | All Threads | Author | Date
All threads index page 1

QSA-interest Archive, April 2007
New bug introduced by a fix in qsa-1.1.5

• Subject: New bug introduced by a fix in qsa-1.1.5
• From: BH <singularita@xxxxxxxxx>
• Date: Thu, 12 Apr 2007 20:29:56 +0200
• Delivered-to: qsa-interest@trolltech.com
• Dkim-signature: a=rsa-sha1; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:mime-version:content-type; b=fQ0uzt1wOwFNuXzXcD4pO1q4y74A5gpyqKfNw3vRbempShuLTT+fxtgPhtka4Sd1Trc7N0V+uAWt3DhGACpVPlgwnsVaT0xniuo2Uoh0P5eFIdyM0i8HWfuLMeth/bWkSvdbzm4eSryTGdZBZri5nPZ0hBL+v8qZTWaGQrKDSMQ=
• Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:mime-version:content-type; b=V4YBiuAgg248nsYhFD43dlejQ2yLfuQXBLBjFZzdrBGNCaHj6706gIy2RNWv/4dM6cJiJ8leekjSQRgr843/3nBOVhmVJy53BsQDy2pZIpTFPeH3zRty5kwYYnHnkncGXW5sx3zxgjhz5M/Yzxm34e5rQml4j0vfC2885VjKgUM=
• List-help: <mailto:qsa-interest-request@trolltech.com?subject=help>
• List-post: <mailto:qsa-interest@trolltech.com>
• List-subscribe: <mailto:qsa-interest-request@trolltech.com?subject=subscribe>
• List-unsubscribe: <mailto:qsa-interest-request@trolltech.com?subject=unsubscribe>
• Resent-from: qsa-interest@xxxxxxxxxxxxx
• Resent-message-id: <O4G98C.A.BV.pqnHGB@esparsett>
• Resent-sender: qsa-interest-request@xxxxxxxxxxxxx
• To: qsa-interest@xxxxxxxxxxxxx

I upgraded from QSA 1.1.4 to QSA 1.1.5
I noticed one new bug was introduced - when calling slots of child
QObject in scripting, it said "No matching overload ...", however with
QSA 1.1.4 the exactly same code worked fine.

In my case, I have class gui::Base as "context" in QSInterpreter and
instance of gui::PageSpace (which is descendant of QWidget) presented
in scripting under name of "PageSpace".

When I called "PageSpace.refresh(document.getFirstPage(),document);"
in script it worked with qsa-1.1.4 and not worked (no matching
overload...) with qsa-1.1.5, though the matching overloads get listed
correctly.

I tracked down the issue to the code in kernel/quickobjects.cpp
In function qsToUObject (which is called form executeSlot) there were
some QS::isNaN tests added.
When I removed these tests (with patch I attached), everything started
to work fine

I guess these tests should fix the "Calling a slot with signature
(int) now reports an error if a string that is not a numeric string,
is passed as argument.", but they break calling other slots.

The patch reintroduces the bug, but it works ... I hope for some
better fix in 1.1.6 :)

These are the relevant slots of "PageSpace" QObject (stripped from
http://pdfedit.cvs.sourceforge.net/pdfedit/pdfedit/src/gui/pagespace.h?view=markup)
in which I noticed this bug:

void refresh ( QSPage * pageToView = NULL, QSPdf * pdf = NULL );
void refresh ( QSPage * pageToView,  QObject * pdf );	
void refresh ( int pageToView, QSPdf * pdf = NULL );	
void refresh ( int pageToView, QObject * pdf );

Hope this will help in identifying and fixing the bug correctly :)

Martin Petricek

Attachment:

Attachment: qsa-1.1.5-bugfix.patch
Description: Binary data

BH wrote:
> I upgraded from QSA 1.1.4 to QSA 1.1.5
> I noticed one new bug was introduced - when calling slots of child
> QObject in scripting, it said "No matching overload ...", however with
> QSA 1.1.4 the exactly same code worked fine.
> 
> In my case, I have class gui::Base as "context" in QSInterpreter and
> instance of gui::PageSpace (which is descendant of QWidget) presented
> in scripting under name of "PageSpace".
> 
> When I called "PageSpace.refresh(document.getFirstPage(),document);"
> in script it worked with qsa-1.1.4 and not worked (no matching
> overload...) with qsa-1.1.5, though the matching overloads get listed
> correctly.

Hi Martin,

I can't seem to reproduce this. Do you have a small compilable example I 
could use?

best regards,
Gunnar

To unsubscribe - send "unsubscribe" in the subject to qsa-interest-request@xxxxxxxxxxxxx