Thread: Attn plugin developers: calibre moving to Qt 5
View Single Post
Old 07-08-2014, 07:01 PM   #18
davidfor
Grand Sorcerer
davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.davidfor ought to be getting tired of karma fortunes by now.
 
Posts: 24,905
Karma: 47303824
Join Date: Jul 2011
Location: Sydney, Australia
Device: Kobo:Touch,Glo, AuraH2O, GloHD,AuraONE, ClaraHD, Libra H2O; tolinoepos
Quote:
Originally Posted by jackie_w View Post
@davidfor,

Oops, posted just before I saw your PM. Could be in for a l-o-n-g night.
It was already a long night for me. I sent the PM and headed to bed.
Quote:
ETA: ... or maybe not Thank you, David

It seems removing the first red line and changing the second to
Code:
action.triggered.connect(slot)
is all that was needed.

Ah, the perils of having only a tenuous grasp on the whole concept of signals and slots and hoping for the best when blindly copying code from other people.
Isn't that how everyone writes code? For this, I had to go and do some reading on it and then I suddenly realised what it was. I like this style a lot more, especially in the windowing code.
davidfor is offline   Reply With Quote