-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Disable GL VU-Meters on windows by default. #11787
Conversation
…terGL to override the default.
} else if (QString::fromLocal8Bit(argv[i]).contains( | ||
"--enableVuMeterGL", Qt::CaseInsensitive)) { | ||
m_useVuMeterGL = true; |
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.
these are hidden from -h
and so forth, is that intentional?
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.
lgtm with the small const fix, let's get this in
This fixes #11789 |
Co-authored-by: Swiftb0y <12380386+Swiftb0y@users.noreply.github.com>
Done. |
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.
LGTM, just fyi there'll probably merge conflicts because we migrated to QCommandlineparser in 2.4 iirc.
In 2.4 we have decided to make "false" default for all. I have merged 2.3 into 2.4 and reverted this PR during resolving of conflicts. |
This fixes: #11785
And adds the command line option --enableVuMeterGL to override the new default.