mirror of
https://gitlab.freedesktop.org/gstreamer/gstreamer.git
synced 2025-03-30 12:49:40 +00:00
validate: Add more warning flags
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/8229>
This commit is contained in:
parent
5c2c1bce6c
commit
f20ecfa920
2 changed files with 16 additions and 8 deletions
|
@ -127,20 +127,27 @@ if gst_debug_disabled and cc.has_argument('-Wno-unused')
|
||||||
add_project_arguments('-Wno-unused', language: 'c')
|
add_project_arguments('-Wno-unused', language: 'c')
|
||||||
endif
|
endif
|
||||||
|
|
||||||
|
# NOTE: Keep entries alphabetically sorted
|
||||||
warning_flags = [
|
warning_flags = [
|
||||||
'-Wmissing-declarations',
|
'-Waddress',
|
||||||
'-Wmissing-prototypes',
|
'-Waggregate-return',
|
||||||
'-Wredundant-decls',
|
|
||||||
'-Wundef',
|
|
||||||
'-Wwrite-strings',
|
|
||||||
'-Wformat',
|
'-Wformat',
|
||||||
'-Wformat-security',
|
'-Wformat-security',
|
||||||
|
'-Wimplicit-fallthrough=3',
|
||||||
'-Winit-self',
|
'-Winit-self',
|
||||||
|
'-Wmissing-declarations',
|
||||||
'-Wmissing-include-dirs',
|
'-Wmissing-include-dirs',
|
||||||
'-Waddress',
|
'-Wmissing-parameter-type',
|
||||||
|
'-Wmissing-prototypes',
|
||||||
'-Wno-multichar',
|
'-Wno-multichar',
|
||||||
'-Wvla',
|
'-Wold-style-definition',
|
||||||
'-Wpointer-arith',
|
'-Wpointer-arith',
|
||||||
|
'-Wredundant-decls',
|
||||||
|
'-Wshift-negative-value',
|
||||||
|
'-Wtype-limits',
|
||||||
|
'-Wundef',
|
||||||
|
'-Wvla',
|
||||||
|
'-Wwrite-strings',
|
||||||
]
|
]
|
||||||
|
|
||||||
foreach extra_arg : warning_flags
|
foreach extra_arg : warning_flags
|
||||||
|
|
|
@ -5181,7 +5181,7 @@ handle_bus_message (MessageData * d)
|
||||||
case GST_MESSAGE_ERROR:
|
case GST_MESSAGE_ERROR:
|
||||||
is_error = TRUE;
|
is_error = TRUE;
|
||||||
|
|
||||||
/* Passthrough */
|
/* FALLTHROUGH */
|
||||||
case GST_MESSAGE_EOS:
|
case GST_MESSAGE_EOS:
|
||||||
{
|
{
|
||||||
GstValidateAction *stop_action;
|
GstValidateAction *stop_action;
|
||||||
|
@ -7147,6 +7147,7 @@ _action_set_done (GstValidateAction * action)
|
||||||
case GST_VALIDATE_EXECUTE_ACTION_ERROR:
|
case GST_VALIDATE_EXECUTE_ACTION_ERROR:
|
||||||
GST_VALIDATE_REPORT_ACTION (scenario, action,
|
GST_VALIDATE_REPORT_ACTION (scenario, action,
|
||||||
SCENARIO_ACTION_EXECUTION_ERROR, "Action %s failed", action->type);
|
SCENARIO_ACTION_EXECUTION_ERROR, "Action %s failed", action->type);
|
||||||
|
/* FALLTHROUGH */
|
||||||
case GST_VALIDATE_EXECUTE_ACTION_ASYNC:
|
case GST_VALIDATE_EXECUTE_ACTION_ASYNC:
|
||||||
case GST_VALIDATE_EXECUTE_ACTION_IN_PROGRESS:
|
case GST_VALIDATE_EXECUTE_ACTION_IN_PROGRESS:
|
||||||
case GST_VALIDATE_EXECUTE_ACTION_NONE:
|
case GST_VALIDATE_EXECUTE_ACTION_NONE:
|
||||||
|
|
Loading…
Reference in a new issue