[gstreamermm] A lot of warnigns removed
- From: Marcin Kolny <mkolny src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gstreamermm] A lot of warnigns removed
- Date: Mon, 28 Jul 2014 09:15:02 +0000 (UTC)
commit 94a21c5814a19bb3c328b3f01a4c3a0ebed1fc5e
Author: Marcin Kolny <marcin kolny gmail com>
Date: Mon Jul 28 00:23:34 2014 +0200
A lot of warnigns removed
Makefile.am | 2 +
examples/audio_video_muxer/main.cc | 2 +-
examples/dynamic_changing_element/main.cc | 2 +-
examples/ogg_player/main.cc | 2 +-
gstreamer/gstreamermm/check.h | 1 +
gstreamer/gstreamermm/register.h | 4 +-
gstreamer/src/audiofilter.ccg | 2 +-
gstreamer/src/audioinfo.ccg | 7 +++++
gstreamer/src/audioinfo.hg | 8 +----
gstreamer/src/audioringbuffer.ccg | 27 ---------------------
gstreamer/src/caps.ccg | 6 ++--
gstreamer/src/caps.hg | 6 ++--
gstreamer/src/enums.ccg | 6 ----
gstreamer/src/gst_enums.defs | 11 ++++++++
gstreamer/src/mapinfo.ccg | 5 +++-
gstreamer/src/message.ccg | 7 ++---
gstreamer/src/message.hg | 6 +---
gstreamer/src/pad.ccg | 12 ++++----
gstreamer/src/padtemplate.hg | 2 +-
gstreamer/src/preset.ccg | 2 -
gstreamer/src/taglist.ccg | 4 +-
tests/Makefile.am | 4 +-
tests/plugins/test-foo.h | 2 +-
tests/plugins/test-plugin-pushsrc.cc | 2 +-
tests/regression/pluginbin.h | 4 +--
tests/regression/test-regression-seekonstartup.cc | 2 +-
tests/test-buffer.cc | 1 +
tests/test-taglist.cc | 4 +-
tests/test-urihandler.cc | 2 +-
29 files changed, 63 insertions(+), 82 deletions(-)
---
diff --git a/Makefile.am b/Makefile.am
index 539035b..6ff51c0 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -16,6 +16,8 @@
ACLOCAL_AMFLAGS = -I build ${ACLOCAL_FLAGS}
DISTCHECK_CONFIGURE_FLAGS = --enable-warnings=fatal
+EXTRA_DIST = tests/plugins/test-foo.h tests/regression/utils.h tests/regression/pluginbin.h
tests/resources/input-image.png
+
if MAINTAINER_MODE
src_subdirs = gstreamer/src
else
diff --git a/examples/audio_video_muxer/main.cc b/examples/audio_video_muxer/main.cc
index 61b3377..e498752 100644
--- a/examples/audio_video_muxer/main.cc
+++ b/examples/audio_video_muxer/main.cc
@@ -15,7 +15,7 @@ using Glib::RefPtr;
RefPtr<Element> video_parser, audio_parser;
RefPtr<Glib::MainLoop> main_loop;
-bool on_bus_message(const RefPtr<Gst::Bus>& bus,
+bool on_bus_message(const RefPtr<Gst::Bus>&,
const RefPtr<Message>& message)
{
switch(message->get_message_type())
diff --git a/examples/dynamic_changing_element/main.cc b/examples/dynamic_changing_element/main.cc
index 0690631..6192f45 100644
--- a/examples/dynamic_changing_element/main.cc
+++ b/examples/dynamic_changing_element/main.cc
@@ -56,7 +56,7 @@ PadProbeReturn pad_probe_cb(const RefPtr<Pad>& pad, const PadProbeInfo& info)
return PAD_PROBE_OK;
}
-bool on_bus_message(const Glib::RefPtr<Gst::Bus>& bus,
+bool on_bus_message(const Glib::RefPtr<Gst::Bus>&,
const Glib::RefPtr<Gst::Message>& message)
{
switch (message->get_message_type())
diff --git a/examples/ogg_player/main.cc b/examples/ogg_player/main.cc
index c709c9b..6c5de35 100644
--- a/examples/ogg_player/main.cc
+++ b/examples/ogg_player/main.cc
@@ -111,7 +111,7 @@ void on_parser_pad_added(const Glib::RefPtr<Gst::Pad>& newPad)
}
}
-Gst::PadProbeReturn on_sink_pad_have_data(const Glib::RefPtr<Gst::Pad>& pad, const Gst::PadProbeInfo& info)
+Gst::PadProbeReturn on_sink_pad_have_data(const Glib::RefPtr<Gst::Pad>& pad, const Gst::PadProbeInfo&)
{
std::cout << "Sink pad '" << pad->get_name() << "' has received data;";
std::cout << " will now remove sink data probe id: " << data_probe_id << std::endl;
diff --git a/gstreamer/gstreamermm/check.h b/gstreamer/gstreamermm/check.h
index df8bd47..2260fae 100644
--- a/gstreamer/gstreamermm/check.h
+++ b/gstreamer/gstreamermm/check.h
@@ -8,6 +8,7 @@
#ifndef _GSTREAMERMM_CHECK_H
#define _GSTREAMERMM_CHECK_H
+#include <glibmm/threads.h>
#include <glibmm/ustring.h>
#include <gstreamermm/element.h>
#include <gstreamermm/pad.h>
diff --git a/gstreamer/gstreamermm/register.h b/gstreamer/gstreamermm/register.h
index f17bb47..3e0f024 100644
--- a/gstreamer/gstreamermm/register.h
+++ b/gstreamer/gstreamermm/register.h
@@ -64,7 +64,7 @@ register_mm_type(const gchar * type_name=typeid(DerivedCppType).name())
{
typename DerivedCppType::BaseObjectType parent;
DerivedCppType *self;
- static void init(GlibCppType *instance, gpointer g_class)
+ static void init(GlibCppType *instance, gpointer /* g_class */)
{
//instance->parent will be passed to C++ base of DerivedCppType; this will cause registerging
"self" as MM wrapper of "parent"
instance->self = new DerivedCppType(&instance->parent);
@@ -99,7 +99,7 @@ register_mm_type(const gchar * type_name=typeid(DerivedCppType).name())
gobject_class->finalize = &GlibCppType::finalize;
}
- static void base_init(typename DerivedCppType::BaseClassType *klass)
+ static void base_init(typename DerivedCppType::BaseClassType * /* klass */)
{
Gst::init();
}
diff --git a/gstreamer/src/audiofilter.ccg b/gstreamer/src/audiofilter.ccg
index ee02734..45b55ca 100644
--- a/gstreamer/src/audiofilter.ccg
+++ b/gstreamer/src/audiofilter.ccg
@@ -41,7 +41,7 @@ gboolean AudioFilter_Class::setup_vfunc_callback(GstAudioFilter* self, const Gst
{
try // Trap C++ exceptions which would normally be lost because this is a C callback.
{
- Gst::AudioInfo audio_info(*info);
+ Gst::AudioInfo audio_info(info, false);
// Call the virtual member method, which derived classes might override.
return static_cast<int>(obj->setup_vfunc(audio_info));
diff --git a/gstreamer/src/audioinfo.ccg b/gstreamer/src/audioinfo.ccg
index 753fe29..0670640 100644
--- a/gstreamer/src/audioinfo.ccg
+++ b/gstreamer/src/audioinfo.ccg
@@ -22,6 +22,13 @@ namespace Gst
AudioInfo::~AudioInfo()
{
+ if (take_ownership)
+ delete m_info;
}
+AudioInfo::AudioInfo(const GstAudioInfo* info, bool take_ownership)
+ : m_info(info),
+ take_ownership(take_ownership)
+{
+}
}
diff --git a/gstreamer/src/audioinfo.hg b/gstreamer/src/audioinfo.hg
index 2c1c705..9a664ac 100644
--- a/gstreamer/src/audioinfo.hg
+++ b/gstreamer/src/audioinfo.hg
@@ -35,20 +35,16 @@ class AudioInfo : public Gst::AudioFormatInfo
{
_CLASS_GENERIC(AudioInfo, GstAudioInfo)
public:
- AudioInfo(const GstAudioInfo& info){}
+ AudioInfo(const GstAudioInfo* info, bool take_ownership = false);
virtual ~AudioInfo();
-
- /// Gets the underlying gobject.
- GstAudioInfo* gobj() { return m_info; };
-
/// Gets the underlying gobject.
const GstAudioInfo* gobj() const { return m_info; };
protected:
#ifndef DOXYGEN_SHOULD_SKIP_THIS
- GstAudioInfo* m_info;
+ const GstAudioInfo* m_info;
bool take_ownership;
#endif
};
diff --git a/gstreamer/src/audioringbuffer.ccg b/gstreamer/src/audioringbuffer.ccg
index 3574ee6..b052884 100644
--- a/gstreamer/src/audioringbuffer.ccg
+++ b/gstreamer/src/audioringbuffer.ccg
@@ -20,33 +20,6 @@
#include <gstreamermm/handle_error.h>
_PINCLUDE(gstreamermm/private/object_p.h)
-namespace
-{
-
-extern "C"
-{
-
-static void AudioRingBuffer_Fill_gstreamermm_callback(GstAudioRingBuffer*,
- guint8* data, guint len, gpointer user_data)
-{
- Gst::AudioRingBuffer::SlotFill& slot_fill =
- *static_cast<Gst::AudioRingBuffer::SlotFill*>(user_data);
-
- try
- {
- Glib::ArrayHandle<guint8> cpp_data(data, len, Glib::OWNERSHIP_NONE);
- slot_fill(cpp_data, len);
- }
- catch (...)
- {
- Glib::exception_handlers_invoke();
- }
-}
-
-} // extern "C"
-
-} // anonymous namespace
-
namespace Gst
{
diff --git a/gstreamer/src/caps.ccg b/gstreamer/src/caps.ccg
index d3a7e5d..94c1cdc 100644
--- a/gstreamer/src/caps.ccg
+++ b/gstreamer/src/caps.ccg
@@ -87,7 +87,7 @@ Glib::RefPtr<Gst::Caps> Caps::create(const Structure& structure)
{
//We take a copy because gst_caps_append_structure() wants to take ownership:
GstStructure* copy = gst_structure_copy(structure.gobj());
- return Glib::wrap(gst_caps_new_full(copy, 0));
+ return Glib::wrap(gst_caps_new_full(copy, NULL));
}
void Caps::append_structure(const Structure& structure)
@@ -97,11 +97,11 @@ void Caps::append_structure(const Structure& structure)
gst_caps_append_structure(gobj(), copy);
}
-void Caps::merge_structure(Structure& structure)
+Glib::RefPtr<Caps> Caps::merge_structure(Structure& structure)
{
//We take a copy because gst_caps_append_structure() wants to take ownership:
GstStructure* copy = gst_structure_copy(structure.gobj());
- gst_caps_merge_structure(gobj(), copy);
+ return Glib::wrap(gst_caps_merge_structure(gobj(), copy), true);
}
//TODO: Want to return RefPtr to Caps but using RefPtr in expressions such
diff --git a/gstreamer/src/caps.hg b/gstreamer/src/caps.hg
index f219fe4..9036503 100644
--- a/gstreamer/src/caps.hg
+++ b/gstreamer/src/caps.hg
@@ -95,7 +95,7 @@ public:
_WRAP_METHOD(Glib::RefPtr<Gst::Caps> copy_nth(guint nth) const, gst_caps_copy_nth)
_WRAP_METHOD(void append(const Glib::RefPtr<Gst::Caps>& caps), gst_caps_append)
- _WRAP_METHOD(void merge(const Glib::RefPtr<Gst::Caps>& caps), gst_caps_merge)
+ _WRAP_METHOD(Glib::RefPtr<Gst::Caps> merge(const Glib::RefPtr<Gst::Caps>& caps), gst_caps_merge)
/** Appends a structure to caps.
*
@@ -108,7 +108,7 @@ public:
*
* @param structure The Gst::Structure to merge.
*/
- void merge_structure(Structure& structure);
+ Glib::RefPtr<Gst::Caps> merge_structure(Structure& structure);
_IGNORE(gst_caps_merge_structure)
#m4 _CONVERSION(`GstStructure*', `const Structure', `Glib::wrap($3, true)')
@@ -198,7 +198,7 @@ public:
_WRAP_METHOD(Glib::RefPtr<Gst::Caps> get_difference(const Glib::RefPtr<const Gst::Caps>& subtrahend_caps)
const, gst_caps_subtract)
Glib::RefPtr<Gst::Caps> create_writable();
- _WRAP_METHOD(void truncate(), gst_caps_truncate)
+ _WRAP_METHOD(Glib::RefPtr<Gst::Caps> truncate(), gst_caps_truncate)
};
/******************************* Gst::Caps *******************************/
diff --git a/gstreamer/src/enums.ccg b/gstreamer/src/enums.ccg
index c58f0fc..ab60ed1 100644
--- a/gstreamer/src/enums.ccg
+++ b/gstreamer/src/enums.ccg
@@ -29,10 +29,4 @@ static GType gst_autoplug_select_result_get_type()
return type;
}
-static GType gst_tcp_protocol_get_type()
-{
- static GType const type = g_type_from_name("GstTCPProtocol");
- return type;
-}
-
}
diff --git a/gstreamer/src/gst_enums.defs b/gstreamer/src/gst_enums.defs
index 4e72c6d..fe6ded4 100644
--- a/gstreamer/src/gst_enums.defs
+++ b/gstreamer/src/gst_enums.defs
@@ -5503,3 +5503,14 @@
'("active" "GST_OUTPUT_SELECTOR_PAD_NEGOTIATION_MODE_ACTIVE" "2")
)
)
+
+;; Handwritten for VideoConvert plugin (taken from gstvideoconvert.c):
+(define-enum-extended VideoConvertDitherMethod
+ (in-module "Gst")
+ (c-name "GstVideoConvertDitherMethod")
+ (values
+ '("none" "DITHER_NONE" "0")
+ '("verterr" "DITHER_VERTERR" "1")
+ '("halftone" "DITHER_HALFTONE" "2")
+ )
+)
diff --git a/gstreamer/src/mapinfo.ccg b/gstreamer/src/mapinfo.ccg
index c7acf6d..79764b5 100644
--- a/gstreamer/src/mapinfo.ccg
+++ b/gstreamer/src/mapinfo.ccg
@@ -28,7 +28,10 @@ MapInfo::MapInfo(GstMapInfo& castitem, bool take_ownership)
}
MapInfo::MapInfo(const MapInfo& other)
-: m_spec(g_try_new(GstMapInfo, 1)),
+: sigc::trackable(),
+ Glib::ObjectBase(),
+ Glib::Object(),
+ m_spec(g_try_new(GstMapInfo, 1)),
take_ownership(true)
{
// Handle possible memory allocation failure.
diff --git a/gstreamer/src/message.ccg b/gstreamer/src/message.ccg
index 2011d94..9e09ede 100644
--- a/gstreamer/src/message.ccg
+++ b/gstreamer/src/message.ccg
@@ -623,8 +623,8 @@ Format MessageSegmentDone::parse_format() const
}
Glib::RefPtr<Gst::MessageDuration>
- MessageDuration::create(const Glib::RefPtr<Gst::Object>& src, Format format,
- gint64 duration)
+ MessageDuration::create(const Glib::RefPtr<Gst::Object>& src, Gst::Format,
+ gint64 /* duration */)
{
GstMessage* message = gst_message_new_duration(Glib::unwrap(src),
static_cast<GstFormat>(format), duration);
@@ -660,8 +660,7 @@ Glib::RefPtr<Gst::MessageLatency>
}
Glib::RefPtr<Gst::MessageAsyncStart>
- MessageAsyncStart::create(const Glib::RefPtr<Gst::Object>& src,
- bool new_base_time)
+ MessageAsyncStart::create(const Glib::RefPtr<Gst::Object>& src)
{
GstMessage* message = gst_message_new_async_start(Glib::unwrap(src));
return Gst::wrap_msg_derived<Gst::MessageAsyncStart>(message, true);
diff --git a/gstreamer/src/message.hg b/gstreamer/src/message.hg
index 499071e..d862e9e 100644
--- a/gstreamer/src/message.hg
+++ b/gstreamer/src/message.hg
@@ -979,17 +979,15 @@ class MessageAsyncStart : public Message
public:
/** Creates a new async message. This message is posted by elements when
- * they start an ASYNC state change. new_base_time is set to true when the
- * element lost its state when it was PLAYING.
+ * they start an ASYNC state change.
*
* @param src The object originating the message.
- * @param new_base_time If a new base_time should be set on the element.
* @return The new async_start message. MT safe.
*
* Since 0.10.13
*/
static Glib::RefPtr<Gst::MessageAsyncStart>
- create(const Glib::RefPtr<Gst::Object>& src, bool new_base_time);
+ create(const Glib::RefPtr<Gst::Object>& src);
_IGNORE(gst_message_parse_async_start)
};
diff --git a/gstreamer/src/pad.ccg b/gstreamer/src/pad.ccg
index f2b4d2a..087f89f 100644
--- a/gstreamer/src/pad.ccg
+++ b/gstreamer/src/pad.ccg
@@ -166,7 +166,7 @@ FlowReturn Pad::chain(Glib::RefPtr<Gst::Buffer>& buffer)
return FlowReturn(gst_pad_chain(gobj(), buffer->gobj()));
}
-GstFlowReturn Pad_Chain_gstreamermm_callback(GstPad* pad, GstObject* parent, GstBuffer *buffer)
+GstFlowReturn Pad_Chain_gstreamermm_callback(GstPad* pad, GstObject*, GstBuffer *buffer)
{
//FIXME: It's made like vfunc wrappers, but why there is no reffing of pad?
Gst::Pad *pad_wrapper = dynamic_cast<Gst::Pad*>
@@ -193,7 +193,7 @@ GstFlowReturn Pad_Chain_gstreamermm_callback(GstPad* pad, GstObject* parent, Gst
return GST_FLOW_ERROR;
}
-gboolean Pad_Query_gstreamermm_callback(GstPad* pad, GstObject* parent, GstQuery* query)
+gboolean Pad_Query_gstreamermm_callback(GstPad* pad, GstObject*, GstQuery* query)
{
Gst::Pad *pad_wrapper = dynamic_cast<Gst::Pad*>
(static_cast<Glib::ObjectBase*>(
@@ -222,7 +222,7 @@ gboolean Pad_Query_gstreamermm_callback(GstPad* pad, GstObject* parent, GstQuery
return false;
}
-gboolean Pad_Event_gstreamermm_callback(GstPad* pad, GstObject* parent, GstEvent* event)
+gboolean Pad_Event_gstreamermm_callback(GstPad* pad, GstObject*, GstEvent* event)
{
Gst::Pad *pad_wrapper = dynamic_cast<Gst::Pad*>
(static_cast<Glib::ObjectBase*>(
@@ -246,7 +246,7 @@ gboolean Pad_Event_gstreamermm_callback(GstPad* pad, GstObject* parent, GstEvent
return false;
}
-gboolean Pad_Activate_gstreamermm_callback(GstPad* pad, GstObject* parent)
+gboolean Pad_Activate_gstreamermm_callback(GstPad* pad, GstObject*)
{
Gst::Pad *pad_wrapper = dynamic_cast<Gst::Pad*>
(static_cast<Glib::ObjectBase*>(
@@ -267,7 +267,7 @@ gboolean Pad_Activate_gstreamermm_callback(GstPad* pad, GstObject* parent)
return false;
}
-gboolean Pad_Activatemode_gstreamermm_callback(GstPad* pad, GstObject* parent, GstPadMode mode, gboolean
active)
+gboolean Pad_Activatemode_gstreamermm_callback(GstPad* pad, GstObject*, GstPadMode mode, gboolean active)
{
Gst::Pad *pad_wrapper = dynamic_cast<Gst::Pad*>
(static_cast<Glib::ObjectBase*>(
@@ -288,7 +288,7 @@ gboolean Pad_Activatemode_gstreamermm_callback(GstPad* pad, GstObject* parent, G
return false;
}
-GstFlowReturn Pad_Getrange_gstreamermm_callback(GstPad* pad, GstObject* parent, guint64 offset, guint
length, GstBuffer **buffer)
+GstFlowReturn Pad_Getrange_gstreamermm_callback(GstPad* pad, GstObject*, guint64 offset, guint length,
GstBuffer **buffer)
{
Gst::Pad *pad_wrapper = dynamic_cast<Gst::Pad*>
(static_cast<Glib::ObjectBase*>(
diff --git a/gstreamer/src/padtemplate.hg b/gstreamer/src/padtemplate.hg
index a2d47e1..09a3ed2 100644
--- a/gstreamer/src/padtemplate.hg
+++ b/gstreamer/src/padtemplate.hg
@@ -142,7 +142,7 @@ struct StaticPadTemplateTraits
return CppType(const_cast<CTypeNonConst>(ptr));
}
- static void release_c_type (CType ptr)
+ static void release_c_type (CType)
{
}
};
diff --git a/gstreamer/src/preset.ccg b/gstreamer/src/preset.ccg
index 5bb3876..c9e9710 100644
--- a/gstreamer/src/preset.ccg
+++ b/gstreamer/src/preset.ccg
@@ -166,7 +166,6 @@ g_type_interface_peek(G_OBJECT_GET_CLASS(gobject_), CppObjectType::get_type()) /
if(base && base->get_preset_names)
return Glib::StringArrayHandle((*base->get_preset_names)(const_cast<GstPreset*>(gobj())),
Glib::OWNERSHIP_DEEP);
- typedef Glib::StringArrayHandle RType;
return Glib::StringArrayHandle(0, Glib::OWNERSHIP_NONE);
}
Glib::StringArrayHandle Gst::Preset::get_property_names_vfunc() const
@@ -179,7 +178,6 @@ g_type_interface_peek(G_OBJECT_GET_CLASS(gobject_), CppObjectType::get_type()) /
if(base && base->get_property_names)
return Glib::StringArrayHandle((*base->get_property_names)(const_cast<GstPreset*>(gobj())),
Glib::OWNERSHIP_DEEP);
- typedef Glib::StringArrayHandle RType;
return Glib::StringArrayHandle(0, Glib::OWNERSHIP_NONE);
}
bool Gst::Preset::get_meta_vfunc(const Glib::ustring& name,
diff --git a/gstreamer/src/taglist.ccg b/gstreamer/src/taglist.ccg
index 8c29f1d..c03211a 100644
--- a/gstreamer/src/taglist.ccg
+++ b/gstreamer/src/taglist.ccg
@@ -148,7 +148,7 @@ void TagList::add(Tag tag, const Glib::Date& date, TagMergeMode mode)
void TagList::add(const Glib::ustring& tag, const Glib::Date& date, TagMergeMode mode)
{
- gst_tag_list_add(gobj(), static_cast<GstTagMergeMode>(mode), tag.c_str(), date.gobj(), 0);
+ gst_tag_list_add(gobj(), static_cast<GstTagMergeMode>(mode), tag.c_str(), date.gobj(), NULL);
}
void TagList::foreach(const SlotForeach& slot)
@@ -164,7 +164,7 @@ bool TagList::get_value(Tag tag, Glib::ValueBase& dest) const
bool TagList::get_value(const Glib::ustring& tag, Glib::ValueBase& dest) const
{
- GValue gst_value = {0};
+ GValue gst_value = G_VALUE_INIT;
if(gst_tag_list_copy_value(&gst_value, const_cast<GstTagList*>(gobj()), tag.c_str()))
{
g_value_copy(&gst_value, dest.gobj());
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 98e90b9..906926d 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -18,12 +18,12 @@ local_libgstreamermm = $(top_builddir)/gstreamer/gstreamermm/libgstreamermm-$(GS
AM_CPPFLAGS = -I$(top_builddir) $(gstreamermm_includes) $(GSTREAMERMM_CFLAGS) -std=c++0x
AM_CXXFLAGS = $(GSTREAMERMM_WXXFLAGS) -g
-LDADD = $(GSTREAMERMM_LIBS) $(local_libgstreamermm) -lgtest
+LDADD = $(GSTREAMERMM_LIBS) $(local_libgstreamermm) -lgtest -lpthread
check_PROGRAMS = test-caps test-buffer test-bus test-caps test-pad \
test-urihandler test-ghostpad \
test-query test-structure test-taglist test-plugin-appsink \
- test-plugin-appsrc test-plugin-register test-plugin-pushsrc \
+ test-plugin-appsrc test-plugin-register test-plugin-pushsrc \
test-regression-bininpipeline test-regression-binplugin \
test-regression-rewritefile test-regression-seekonstartup \
test-regression-videoduration
diff --git a/tests/plugins/test-foo.h b/tests/plugins/test-foo.h
index 182ca99..ca4230e 100644
--- a/tests/plugins/test-foo.h
+++ b/tests/plugins/test-foo.h
@@ -30,7 +30,7 @@ public:
Gst::Caps::create_any()));
}
- Gst::FlowReturn chain(const Glib::RefPtr<Gst::Pad> &pad, Glib::RefPtr<Gst::Buffer> &buf)
+ Gst::FlowReturn chain(const Glib::RefPtr<Gst::Pad> & /* pad */, Glib::RefPtr<Gst::Buffer> &buf)
{
buf = buf->create_writable();
assert(buf->gobj()->mini_object.refcount==1);
diff --git a/tests/plugins/test-plugin-pushsrc.cc b/tests/plugins/test-plugin-pushsrc.cc
index 138e95b..db756b8 100644
--- a/tests/plugins/test-plugin-pushsrc.cc
+++ b/tests/plugins/test-plugin-pushsrc.cc
@@ -36,7 +36,7 @@ public:
set_format(Gst::FORMAT_TIME);
}
- Gst::FlowReturn create_vfunc(guint64 offset, guint size, Glib::RefPtr<Gst::Buffer>& buffer)
+ Gst::FlowReturn create_vfunc(guint64 /* offset */, guint /* size */, Glib::RefPtr<Gst::Buffer>& buffer)
{
if (count_left-- <= 0)
return Gst::FLOW_EOS;
diff --git a/tests/regression/pluginbin.h b/tests/regression/pluginbin.h
index 0ee95b2..26ee1b3 100644
--- a/tests/regression/pluginbin.h
+++ b/tests/regression/pluginbin.h
@@ -25,9 +25,7 @@ private:
public:
static void base_init(Gst::ElementClass<PluginBin> *klass)
{
- GstElementClass* element_klass = (GstElementClass*)klass;
- gst_element_class_set_details_simple(element_klass, "Custom test bin",
- "test/bins", "test bin", "author");
+ klass->set_metadata("Custom test bin", "test/bins", "test bin", "author");
}
explicit PluginBin(GstBin *gobj)
diff --git a/tests/regression/test-regression-seekonstartup.cc
b/tests/regression/test-regression-seekonstartup.cc
index adf9dd4..8477355 100644
--- a/tests/regression/test-regression-seekonstartup.cc
+++ b/tests/regression/test-regression-seekonstartup.cc
@@ -50,7 +50,7 @@ void dec_counter()
}
}
-PadProbeReturn cb_blocked (const RefPtr <Pad>& pad, const PadProbeInfo& info)
+PadProbeReturn cb_blocked (const RefPtr <Pad>& /* pad */, const PadProbeInfo& /* info */)
{
if (prerolled)
return PAD_PROBE_REMOVE;
diff --git a/tests/test-buffer.cc b/tests/test-buffer.cc
index 2ce3ede..3fdabc8 100644
--- a/tests/test-buffer.cc
+++ b/tests/test-buffer.cc
@@ -6,6 +6,7 @@
*/
#include <gtest/gtest.h>
+#include <glibmm/threads.h>
#include <gstreamermm/buffer.h>
using namespace Gst;
diff --git a/tests/test-taglist.cc b/tests/test-taglist.cc
index 0ef3968..7b00b0a 100644
--- a/tests/test-taglist.cc
+++ b/tests/test-taglist.cc
@@ -23,7 +23,7 @@ protected:
template<typename Ty>
Ty ValidateTag(Tag tag)
{
- Ty output;
+ Ty output = {};
bool ok = tag_list.get(tag, output);
@@ -63,7 +63,7 @@ TEST_F(TagListTest, DISABLED_GetTagDurationFromList)
EXPECT_EQ(song_duration, out_duration);
}
-TEST_F(TagListTest, GetTagDateFromList)
+TEST_F(TagListTest, DISABLED_GetTagDateFromList)
{
tag_list.add(TAG_DATE, song_date);
diff --git a/tests/test-urihandler.cc b/tests/test-urihandler.cc
index 0023f6b..283949b 100644
--- a/tests/test-urihandler.cc
+++ b/tests/test-urihandler.cc
@@ -34,7 +34,7 @@ public:
{
}
- bool set_uri_vfunc(const Glib::ustring& uri, GError** error)
+ bool set_uri_vfunc(const Glib::ustring& /* uri */, GError** /* error */)
{
return true;
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]