New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update controllerdebug.h to unqouted output for mixxx.log #1853
Conversation
Thank you! Could you please sign the Mixxx contributor agreement? This gives us permissions to distribute your change. |
@@ -33,7 +33,7 @@ class ControllerDebug { | |||
#define controllerDebug(stream) \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if you could add a comment above this line mentioning Bug #1797746 and why the noquote is present that would be nice to avoid accidental removals
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done! sorry for my poor english.
Please rebase on the 2.2 branch and retarget this PR to 2.2. Switching the target branch of this PR didn't work as expected. |
25588ae
to
8e75f0c
Compare
Fix for https://bugs.launchpad.net/mixxx/+bug/1797746 Without this patch, debugging messages in mixxx.log are quoted since QT > 5.4, which breaks output of engine.log() escape characters. This patch adds .noquote() to QDebug if QT_VERSION >= 5.4
Thank you for rebasing. @banad60 can you confirm that you have signed the contributor agreement linked above? |
Since qt5, QDebug sems quoted by default. But for debugging we need unquoted output in mixxx.log