vaapi: Fix various compiler warnings and disable -Wredundant-decls for now

This commit is contained in:
Sebastian Dröge 2016-02-16 15:09:01 +02:00
parent 7bd9b2aa5c
commit 734b8bad8f
8 changed files with 12 additions and 11 deletions

View file

@ -837,7 +837,8 @@ dnl make sure it doesn't complain about unused variables if debugging is disable
AG_GST_CHECK_GST_DEBUG_DISABLED([NO_WARNINGS="-Wno-unused"], [NO_WARNINGS=""]) AG_GST_CHECK_GST_DEBUG_DISABLED([NO_WARNINGS="-Wno-unused"], [NO_WARNINGS=""])
dnl define an ERROR_CFLAGS Makefile variable dnl define an ERROR_CFLAGS Makefile variable
AG_GST_SET_ERROR_CFLAGS([$FATAL_WARNINGS], [-Wmissing-declarations -Wmissing-prototypes -Wredundant-decls -Wwrite-strings -Wformat-security -Wold-style-definition -Winit-self -Wmissing-include-dirs -Waddress -Wno-multichar -Wnested-externs $NO_WARNINGS]) dnl FIXME Add -Wredundant-decls again if considered useful and warnings are fixed
AG_GST_SET_ERROR_CFLAGS([$FATAL_WARNINGS], [-Wmissing-declarations -Wmissing-prototypes -Wwrite-strings -Wformat-security -Wold-style-definition -Winit-self -Wmissing-include-dirs -Waddress -Wno-multichar -Wnested-externs $NO_WARNINGS])
dnl define correct level for debugging messages dnl define correct level for debugging messages
AG_GST_SET_LEVEL_DEFAULT([$GST_GIT]) AG_GST_SET_LEVEL_DEFAULT([$GST_GIT])

View file

@ -365,7 +365,7 @@ copy_quant_matrix (guint8 dst[64], const guint8 src[64])
static const char * static const char *
get_profile_str (GstVaapiProfile profile) get_profile_str (GstVaapiProfile profile)
{ {
char *str; const char *str;
switch (profile) { switch (profile) {
case GST_VAAPI_PROFILE_MPEG2_SIMPLE: case GST_VAAPI_PROFILE_MPEG2_SIMPLE:

View file

@ -456,7 +456,7 @@ decode_gop (GstVaapiDecoderMpeg4 * decoder, const guint8 * buf, guint buf_size)
return GST_VAAPI_DECODER_STATUS_SUCCESS; return GST_VAAPI_DECODER_STATUS_SUCCESS;
} }
void static void
calculate_pts_diff (GstVaapiDecoderMpeg4 * decoder, calculate_pts_diff (GstVaapiDecoderMpeg4 * decoder,
GstMpeg4VideoObjectLayer * vol_hdr, GstMpeg4VideoObjectPlane * vop_hdr) GstMpeg4VideoObjectLayer * vol_hdr, GstMpeg4VideoObjectPlane * vop_hdr)
{ {

View file

@ -572,7 +572,7 @@ decode_buffer (GstVaapiDecoderVp8 * decoder, const guchar * buf, guint buf_size)
return decode_picture (decoder, buf, buf_size); return decode_picture (decoder, buf, buf_size);
} }
GstVaapiDecoderStatus static GstVaapiDecoderStatus
gst_vaapi_decoder_vp8_decode (GstVaapiDecoder * base_decoder, gst_vaapi_decoder_vp8_decode (GstVaapiDecoder * base_decoder,
GstVaapiDecoderUnit * unit) GstVaapiDecoderUnit * unit)
{ {

View file

@ -647,7 +647,7 @@ decode_buffer (GstVaapiDecoderVp9 * decoder, const guchar * buf, guint buf_size)
return decode_picture (decoder, buf, size); return decode_picture (decoder, buf, size);
} }
GstVaapiDecoderStatus static GstVaapiDecoderStatus
gst_vaapi_decoder_vp9_decode (GstVaapiDecoder * base_decoder, gst_vaapi_decoder_vp9_decode (GstVaapiDecoder * base_decoder,
GstVaapiDecoderUnit * unit) GstVaapiDecoderUnit * unit)
{ {

View file

@ -79,7 +79,7 @@ struct _GstVaapiFilter
/* --- VPP Types --- */ /* --- VPP Types --- */
/* ------------------------------------------------------------------------- */ /* ------------------------------------------------------------------------- */
GType static GType
gst_vaapi_scale_method_get_type (void) gst_vaapi_scale_method_get_type (void)
{ {
static gsize g_type = 0; static gsize g_type = 0;
@ -725,7 +725,7 @@ ensure_operations (GstVaapiFilter * filter)
} }
/* Find whether the VPP operation is supported or not */ /* Find whether the VPP operation is supported or not */
GstVaapiFilterOpData * static GstVaapiFilterOpData *
find_operation (GstVaapiFilter * filter, GstVaapiFilterOp op) find_operation (GstVaapiFilter * filter, GstVaapiFilterOp op)
{ {
guint i; guint i;
@ -905,7 +905,7 @@ op_set_deinterlace (GstVaapiFilter * filter, GstVaapiFilterOpData * op_data,
/* Update skin tone enhancement */ /* Update skin tone enhancement */
#if USE_VA_VPP #if USE_VA_VPP
gboolean static gboolean
op_set_skintone_unlocked (GstVaapiFilter * filter, op_set_skintone_unlocked (GstVaapiFilter * filter,
GstVaapiFilterOpData * op_data, gboolean value) GstVaapiFilterOpData * op_data, gboolean value)
{ {

View file

@ -461,7 +461,7 @@ gst_vaapi_decode_bin_class_init (GstVaapiDecodeBinClass * klass)
static gboolean static gboolean
gst_vaapi_decode_bin_configure (GstVaapiDecodeBin * vaapidecbin) gst_vaapi_decode_bin_configure (GstVaapiDecodeBin * vaapidecbin)
{ {
gchar *missing_factory = NULL; const gchar *missing_factory = NULL;
GstPad *pad, *ghostpad; GstPad *pad, *ghostpad;
GstPadTemplate *tmpl; GstPadTemplate *tmpl;

View file

@ -358,7 +358,7 @@ gst_vaapi_value_set_format_list (GValue * value, GArray * formats)
return TRUE; return TRUE;
} }
void static void
set_video_template_caps (GstCaps * caps) set_video_template_caps (GstCaps * caps)
{ {
GstStructure *const structure = gst_caps_get_structure (caps, 0); GstStructure *const structure = gst_caps_get_structure (caps, 0);
@ -662,7 +662,7 @@ gst_video_info_change_format (GstVideoInfo * vip, GstVideoFormat format,
* gst_vaapi_display_unref () after use. * gst_vaapi_display_unref () after use.
**/ **/
GstVaapiDisplay * GstVaapiDisplay *
gst_vaapi_create_test_display () gst_vaapi_create_test_display (void)
{ {
return gst_vaapi_create_display (GST_VAAPI_DISPLAY_TYPE_ANY, NULL); return gst_vaapi_create_display (GST_VAAPI_DISPLAY_TYPE_ANY, NULL);
} }