Exverge/suyu
3
0
Fork 0
forked from suyu/suyu
Code Pull requests Activity

yuzu/CMakeLists: Disable unsafe overloads of QProcess' start() function

Browse source 
Other overloads of start() are considerably much safer to use if we ever
need this in the future and need to pass arguments to the program, given
it contains separate parameters for the program path and the arguments
themselves, whereas this unsafe overload contains both as a single
string.

Given the alternatives are much safer, we can disable this.
This commit is contained in:
Lioncash 2019-06-05 15:47:42 -04:00
parent b5e1e87922
commit 5b93290183
1 changed files with 3 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/yuzu/CMakeLists.txt
View file

@ -162,6 +162,9 @@ target_compile_definitions(yuzu PRIVATE
# Disable implicit type narrowing in signal/slot connect() calls. # Disable implicit type narrowing in signal/slot connect() calls.
-DQT_NO_NARROWING_CONVERSIONS_IN_CONNECT -DQT_NO_NARROWING_CONVERSIONS_IN_CONNECT
# Disable unsafe overloads of QProcess' start() function.
-DQT_NO_PROCESS_COMBINED_ARGUMENT_START
) )
if (YUZU_ENABLE_COMPATIBILITY_REPORTING) if (YUZU_ENABLE_COMPATIBILITY_REPORTING)