pcp
[Top] [All Lists]

Re: kmchart updates

To: nscott@xxxxxxxxxx
Subject: Re: kmchart updates
From: Olivier Tarnus <o.tarnus@xxxxxxx>
Date: Wed, 17 Oct 2007 01:44:59 +0200
Cc: pcp@xxxxxxxxxxx
In-reply-to: <1192577104.5414.45.camel@xxxxxxxxxxxxxx>
References: <1192520068.5414.38.camel@xxxxxxxxxxxxxx> <471522C1.9040009@xxxxxxx> <1192577104.5414.45.camel@xxxxxxxxxxxxxx>
Sender: pcp-bounce@xxxxxxxxxxx
User-agent: Thunderbird 2.0.0.6 (X11/20070806)
Hey...

Had some time to spend tonight :-)

So i fixed calls to use triggered(), and also made some fix for
InfoDialog::infoTabCurrentChanged(QWidget*) and
KmChart::activeTabChanged(QWidget*) as they had also some problems
because of the change of QTabWidget::currentChanged to int.

Everything now runs fine... Happy to have fixed it myself ;-)

Olivier


Nathan Scott wrote:
> On Tue, 2007-10-16 at 22:44 +0200, Olivier Tarnus wrote:
>   
>> Hi Nathan,
>>
>>     
>
> Hi there Olivier,
>
>   
>> Seeing all the activity on kmchart in the recent times, I decided to
>> try it.
>>
>> It built perfectly on gentoo linux x86_64 with qt-4.3.1, but i get
>> errors at runtime and it seems that widget actions are not caught,
>> see:
>>     
>
>   
>> -- snip --
>> tom@darkstar ~/dev/kmchart/src $ kmchart
>> Object::connect: No such signal QAction::activated()
>> Object::connect:  (sender name:   'fileNewChartAction')
>> Object::connect:  (receiver name: 'KmChart')
>>     
>
> Ah, I see the problem - we're using the activated() signal, which is
> only supported in QT4 if QT3_SUPPORT is defined in the headers; from
> qaction.h...
>
> Q_SIGNALS:
>     void changed();
>     void triggered(bool checked = false);
>     void hovered();
>     void toggled(bool);
> #ifdef QT3_SUPPORT
>     QT_MOC_COMPAT void activated(int = 0);
> #endif
>
> Looks like "triggered() is the prefered way to do this on QT4, I'll
> switch over to that.
>
>   
>> After that nothing works, ie: no action on the gui will produce any
>> effect, even the quit menu item.
>>     
>
> Yeah, thats kinda what I'd expect if that signal wasn't hooked up.
>
>   
>> Perhaps not the best place to talk about that, but i've not been able
>> to
>> find any website or list dedicated to kmchart.
>>     
>
> This is the right place.  I'll commit that fix later this morning, and
> CC you when it goes in - thanks for reporting the problem.
>
> cheers.
>
> --
> Nathan
>
>
>   


<Prev in Thread] Current Thread [Next in Thread>