<table><tr><td style="">lancaster 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/D25105">View Revision</a></tr></table><br /><div><div><p>Ok I think these changes make a lot of sense.  Making a method for checking if guiding is active instead of just checking the state is a very good idea because then we can do what you did and check a number of guide states at the same time, but also if we need to check something else besides the state we could do that too.  So I like that.</p>

<p>I also agree that a lot of that dead code can go, it is taking up room and you have to get around it to find the code that you want to work on.  When I was making my recent changes for PHD2, I was surprised at how much commented out code there was.  But some of it did prove useful actually and I used it for a reference of how what I was trying to do used to be done.  But now, yes, I think it can go.</p>

<p>Part of the dead code you are removing is related to rapid guide, something Jasem had been working on that he said did not work well.  I don't know if he ultimately has decided to abandon it.   If he is fine with having that code removed, then great.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>BRANCH</strong><div><div>meridian_flip_bugfix_restart_guiding</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25105">https://phabricator.kde.org/D25105</a></div></div><br /><div><strong>To: </strong>wreissenberger, mutlaqja, lancaster<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>