D29008: Use the mount Pier Side property to manage Pier Flip
Jasem Mutlaq
noreply at phabricator.kde.org
Wed Apr 22 07:00:40 BST 2020
mutlaqja added inline comments.
INLINE COMMENTS
> TallFurryMan wrote in capture.cpp:6831
> Prototype `char const * Capture::MFStageString(MFStage stage) const` is more appropriate?
> That function has no interest in providing a QString object.
> Also, these strings are used in logs, so you may want to output something more friendly?
Isn't this getting casted to QString? I saw we avoid C strings in the code. Perhaps the only change required here is more friendly names.
REPOSITORY
R321 KStars
REVISION DETAIL
https://phabricator.kde.org/D29008
To: chrisrowland, mutlaqja
Cc: TallFurryMan, wreissenberger, kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20200422/79f42a57/attachment.html>
More information about the kde-edu
mailing list