video_core: Enforce C4715 (not all control paths return a value)

Most of the time people write code that always returns a value,
terminates execution, throws an exception, or uses an unconventional
jump primitive.

This is not always true when we build without asserts on mainline builds.
To avoid introducing undefined behavior on our most used builds, enforce
this warning signalling an error and stopping the build from shipping.
This commit is contained in:
ReinUsesLisp 2020-12-24 21:01:23 -03:00
parent 5dbda22659
commit 1e191cc837

View file

@ -303,6 +303,7 @@ if (MSVC)
/we4457 # Declaration of 'identifier' hides function parameter
/we4458 # Declaration of 'identifier' hides class member
/we4459 # Declaration of 'identifier' hides global declaration
/we4715 # 'function' : not all control paths return a value
)
else()
target_compile_options(video_core PRIVATE