<table><tr><td style="">mutlaqja added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D13769">View Revision</a></tr></table><br /><div><div><p>Since you're updated Signal/Slots, may I suggest you used Qt5 new way for connection? I've been moving toward this in the code to make sure all is good at compile time. I've been bitten many times when some slots name was wrong and sent me on a goose chase to figure out what was going on. So it's like connect(someObject, &SomeClass::signal, anotherObject, &AnotherObject::function)..</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13769">https://phabricator.kde.org/D13769</a></div></div><br /><div><strong>To: </strong>TallFurryMan, mutlaqja<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>