12 years agointrospection: add Value annotations for GST_SECOND, GST_MSECOND, GST_USECOND, GST_NS...
introspection: add Value annotations for GST_SECOND, GST_MSECOND, GST_USECOND, GST_NSECOND constants
gobject-introspection won't parse them properly otherwise.
Still need to force the right type though (either GstClockTime or
guint64), but Type: xyz has no effect for me here, so someone with
a newer g-i needs to test this.
Some other defines are also missing, e.g. GST_CLOCK_TIME_NONE.
gobject-introspection won't parse them properly otherwise.
Still need to force the right type though (either GstClockTime or
guint64), but Type: xyz has no effect for me here, so someone with
a newer g-i needs to test this.
Some other defines are also missing, e.g. GST_CLOCK_TIME_NONE.
configure: suppress unused variable warnings if gst debugging is disabled
https://bugzilla.gnome.org/show_bug.cgi?id=662952
https://bugzilla.gnome.org/show_bug.cgi?id=662952
configure: fix typo around GLIB_EXTRA_CFLAGS in GST_ALL_CXXFLAGS
queue: use statically allocated GQueue
bin: use statically allocated GQueue
Because we can.
Because we can.
collectpads2: use flags enum instead of guint in structure
queue2: use statically allocated GQueue
ghostpad: Don't cache internal proxy pad target
The internal proxy pad target is simply a cache of the internal proxy pad
peer. This patch uses the well implement GstPad peer handling to obtain the
target. This fixes issues with target not being set in both direction when
two ghostpads are linked together (empty bin).
https://bugzilla.gnome.org/show_bug.cgi?id=658517
The internal proxy pad target is simply a cache of the internal proxy pad
peer. This patch uses the well implement GstPad peer handling to obtain the
target. This fixes issues with target not being set in both direction when
two ghostpads are linked together (empty bin).
https://bugzilla.gnome.org/show_bug.cgi?id=658517
adapter: simplify gst_adapter_take_list()
Use a stack-allocated GQueue to assemble our GList.
Use a stack-allocated GQueue to assemble our GList.
configure: make GLIB_EXTRA_CFLAGS overwritable
Make 'make GLIB_EXTRA_CFLAGS=...' work.
Make 'make GLIB_EXTRA_CFLAGS=...' work.
filter: deprecate gst_filter_run()
It's not really used outside of core at all, and has
serious namespace issues. If anyone feels the need to
revive this one, please use a less generic name space.
API: deprecate gst_filter_run()
API: deprecate GstFilterFunc
It's not really used outside of core at all, and has
serious namespace issues. If anyone feels the need to
revive this one, please use a less generic name space.
API: deprecate gst_filter_run()
API: deprecate GstFilterFunc
registry: don't use soon-to-be-deprecated gst_filter_run()
Lines-of-code savings are negligible anyway.
Lines-of-code savings are negligible anyway.
pluginfeature: deprecate gst_plugin_feature_type_name_filter()
It's only used internally anyway and the helper struct
has namespace issues.
API: deprecated gst_plugin_feature_type_name_filter()
API: deprecated GstTypeNameData
It's only used internally anyway and the helper struct
has namespace issues.
API: deprecated gst_plugin_feature_type_name_filter()
API: deprecated GstTypeNameData
taglist: add to_string and new_from_string functions
We want to make GstTagList opaque and not have people use
GstStructure API on it.
API: gst_tag_list_to_string()
API: gst_tag_list_new_from_string()
We want to make GstTagList opaque and not have people use
GstStructure API on it.
API: gst_tag_list_to_string()
API: gst_tag_list_new_from_string()
taglist: avoid pointless tag name -> quark lookups
We never get a tag name quark from a caller, it's always a
string, from which we'll try to look up our tag info in the
hash table, so change the hash table key from quark to string.
Avoids a bunch of pointless string => quark lookup in the
global quark table. We need to do an extra string => quark
conversion now when we copy a taglist, but in that case we're
in a slow path anyway.
We never get a tag name quark from a caller, it's always a
string, from which we'll try to look up our tag info in the
hash table, so change the hash table key from quark to string.
Avoids a bunch of pointless string => quark lookup in the
global quark table. We need to do an extra string => quark
conversion now when we copy a taglist, but in that case we're
in a slow path anyway.
taglist: add gst_tag_list_is_equal()
API: gst_tag_list_is_equal()
API: gst_tag_list_is_equal()
structure: identical structures are definitely equal
basetransform: Also fush the cache when changing the upstream caps suggestion
basetransform: Add debug output when returning a cached transform
collectpads2: Fix refcount handling if a buffer was dropped due to clipping
collectpads2: Merge the clip and prepare_buffer function into one
collectpads2: Merge clipping API from old collectpads
basesink: make default query function name show up in gst-inspect
collectpads2: avoid hanging in case of sparse newsegment events
... in the extent that a non-waiting pad (so indicated by newsegment)
turns out to provide the best buffer, which is then forced to waiting
for book-keeping purposes, but that should only be temporary.
See bug #415754.
... in the extent that a non-waiting pad (so indicated by newsegment)
turns out to provide the best buffer, which is then forced to waiting
for book-keeping purposes, but that should only be temporary.
See bug #415754.
collectpads2: Use G_DEFINE_TYPE instead of GST_BOILERPLATE
collectpads2: Add to the documentation
win32: Add new collectpads2 API
base: Add collectpads2
This handles muxing of sparse/subtitle streams and has
lots of cleanup. Still missing is special support for
live streams but this can be added later without breaking
API/ABI.
Based on the version from the videomixer plugin.
https://bugzilla.gnome.org/show_bug.cgi?id=415754
This handles muxing of sparse/subtitle streams and has
lots of cleanup. Still missing is special support for
live streams but this can be added later without breaking
API/ABI.
Based on the version from the videomixer plugin.
https://bugzilla.gnome.org/show_bug.cgi?id=415754
basetransform: Fix refcount leak
Don't leak peercaps and a ref to the basetransform when returning
the cached caps.
Don't leak peercaps and a ref to the basetransform when returning
the cached caps.
controller: fix wrong order of calls in the docs.
event: clarify docs for step event
registry: add support for GST_REGISTRY_REUSE_PLUGIN_SCANNER=no
This will make sure we spawn a new plugin scanner helper for each plugin
to be introspected, which helps with making sure we don't load too many
shared objects (libs, plugins) at the same time on systems where there
is a hard limit like on Android.
A better version might re-use the scanner for up to N times, though
it's not clear whether that would actually improve things dramatically.
https://bugzilla.gnome.org/show_bug.cgi?id=662091
This will make sure we spawn a new plugin scanner helper for each plugin
to be introspected, which helps with making sure we don't load too many
shared objects (libs, plugins) at the same time on systems where there
is a hard limit like on Android.
A better version might re-use the scanner for up to N times, though
it's not clear whether that would actually improve things dramatically.
https://bugzilla.gnome.org/show_bug.cgi?id=662091
caps: No need to require writable caps for _append/_merge second caps
The second caps ownership is transfered, no need to require it to
be writable from the caller function. Instead, _append and _merge
make it writable on their own.
Discovered because of an assertion on encoding-profile.c in
_get_input_caps using _merge but not passing writable caps.
The second caps ownership is transfered, no need to require it to
be writable from the caller function. Instead, _append and _merge
make it writable on their own.
Discovered because of an assertion on encoding-profile.c in
_get_input_caps using _merge but not passing writable caps.
baseparse: Fix documentation, it's pre_push_frame and not pre_push_buffer
basetransform: cache transformed caps where appropriate
Speeds up negotiation a fair bit on a contrived pipeline
with a dozen colorspace conversions.
Hopefully clears out the cache every time it ought to.
https://bugzilla.gnome.org/show_bug.cgi?id=662291
Speeds up negotiation a fair bit on a contrived pipeline
with a dozen colorspace conversions.
Hopefully clears out the cache every time it ought to.
https://bugzilla.gnome.org/show_bug.cgi?id=662291
12 years agobasebarse: Add detect vfunc to allow subclasses to do format detection before anythin...
basebarse: Add detect vfunc to allow subclasses to do format detection before anything else
API: GstBaseParseClass::detect()
This is called with the first buffers until the subclass has finished detection
and only afterwards the original buffers are handled as before. The vfunc allows
detection of the stream format without breaking the upstream framing.
API: GstBaseParseClass::detect()
This is called with the first buffers until the subclass has finished detection
and only afterwards the original buffers are handled as before. The vfunc allows
detection of the stream format without breaking the upstream framing.
pad: not only describe conditions in the docs, also check them in the code
When blocking pads, check if the pad is in the rigt direction. Log some info
for the developer and return FALSE, instead of just locking up.
When blocking pads, check if the pad is in the rigt direction. Log some info
for the developer and return FALSE, instead of just locking up.
message: protect against null message sources
Message sources can be null, check if it is before trying to
access its name.
Message sources can be null, check if it is before trying to
access its name.
baseparse: add since doc to new getcaps function
logging: more logging and prefer human readable details over memory locations
debugutils: improve display of ghost- and proxypads
Handle virtual links between ghost and proxypads when iterating pads instead of
when linking. Besides using less code this provides a more accurate picture.
Handle virtual links between ghost and proxypads when iterating pads instead of
when linking. Besides using less code this provides a more accurate picture.
baseparse: add getcaps function
Adds a getcaps function to the sink pad to make parsers propagate
downstream caps restrictions to upstream.
The pipeline "audiotestsrc num-buffers=100 ! faac ! aacparse !
"audio/mpeg, version=(int)4, stream-format=(string)adts" ! filesink"
wouldn't work because aacparse wouldn't propagate the adts restriction
upstream to faac.
This patch adds a default getcaps to the sink pad to simply proxy
downstream caps and also adds a 'get_sink_caps' function pointer
to GstBaseParseClass for subclasses that need more refined getcaps.
https://bugzilla.gnome.org/show_bug.cgi?id=661874
Adds a getcaps function to the sink pad to make parsers propagate
downstream caps restrictions to upstream.
The pipeline "audiotestsrc num-buffers=100 ! faac ! aacparse !
"audio/mpeg, version=(int)4, stream-format=(string)adts" ! filesink"
wouldn't work because aacparse wouldn't propagate the adts restriction
upstream to faac.
This patch adds a default getcaps to the sink pad to simply proxy
downstream caps and also adds a 'get_sink_caps' function pointer
to GstBaseParseClass for subclasses that need more refined getcaps.
https://bugzilla.gnome.org/show_bug.cgi?id=661874
basesrc: also update the stream time
baseparse: Fix handling of queued frames
gst_base_parse_push_frame() already frees the frame, no need to
do it another time again.
gst_base_parse_push_frame() already frees the frame, no need to
do it another time again.
bus: give watch source a name
Give our GSource a meaningful name. Source names can be
used for debugging and profiling, for example with
systemtap or gdb.
Give our GSource a meaningful name. Source names can be
used for debugging and profiling, for example with
systemtap or gdb.
gstbuffer: Add transfer annotations for gst_buffer_join()
bus: fix typo in the docs
debugutils: show if an element is state-locked
logging: use _OBJECT variants more
buffer: Use an inline function instead of a macro for gst_buffer_replace()
This gives us type checks by the compiler and more useful compiler errors.
This gives us type checks by the compiler and more useful compiler errors.
event: Use an inline function instead of a macro for gst_event_replace()
This gives us type checks by the compiler and more useful compiler errors.
This gives us type checks by the compiler and more useful compiler errors.
basesrc: properly adjust start time
When we do a non-flushing seek and closed the current segment,
make sure that we open the next segment from where we closed.
When we do a non-flushing seek and closed the current segment,
make sure that we open the next segment from where we closed.
utils: catch invalid instance sizes in gst_type_register_static_full()
Add guards to catch overly large instance sizes.
https://bugzilla.gnome.org/show_bug.cgi?id=660955
Add guards to catch overly large instance sizes.
https://bugzilla.gnome.org/show_bug.cgi?id=660955
gstparse: prefix generated parser functions so they don't get exported
Don't export those 35-something random _gst_parse_yy* symbols. These were
never in any header files and also blacklisted from our .def files, in
case anyone wonders.
Don't export those 35-something random _gst_parse_yy* symbols. These were
never in any header files and also blacklisted from our .def files, in
case anyone wonders.
pipeline: Use pipeline category for one more log message
Makes debugging easier.
Makes debugging easier.
GST_PTR_FORMAT: Add GstBuffer ptr format and use in GST_SCHEDULING
GstBuffer pointers can now be printed using GST_PTR_FORMAT. This is used
in the very useful GST_SCHEDULING debug logs in gstpad.c and allows for
easier and more information tracking of buffer progress through a
pipeline with just debug logging.
GstBuffer pointers can now be printed using GST_PTR_FORMAT. This is used
in the very useful GST_SCHEDULING debug logs in gstpad.c and allows for
easier and more information tracking of buffer progress through a
pipeline with just debug logging.
baseparse: send duration message when updating internal duration
baseparse: Return success if optional start/stop method is not provided
This allows to not implement the optional start/stop methods.
This allows to not implement the optional start/stop methods.
gstquery: Make debugging message more informative
For all the newcomers out there who still don't know the values of
GstQueryType enum by heart...
... and old-timers who've got better things to do :)
For all the newcomers out there who still don't know the values of
GstQueryType enum by heart...
... and old-timers who've got better things to do :)
baseparse: make estimating the position in query handler actually work
No point estimating if we don't set the result afterwards.
No point estimating if we don't set the result afterwards.
baseparse: answer position query in stream time and try upstream first
Let the demuxer have first say as well.
https://bugzilla.gnome.org/show_bug.cgi?id=659485
Let the demuxer have first say as well.
https://bugzilla.gnome.org/show_bug.cgi?id=659485
gst.h: include header for atomic queue
tests: add minimal test for GstAtomicQueue
Just new + free.
Just new + free.
basetransform: send delayed events earlier
Some elements (such as videorate) might push buffers early,
for instance in in transform_ip. We want events (and in particular
any NEWSEGMENT event) to be pushed before that.
This fixes transmageddon wedging on converting a file starting
with a non zero offset to Ogg.
https://bugzilla.gnome.org/show_bug.cgi?id=660165
Some elements (such as videorate) might push buffers early,
for instance in in transform_ip. We want events (and in particular
any NEWSEGMENT event) to be pushed before that.
This fixes transmageddon wedging on converting a file starting
with a non zero offset to Ogg.
https://bugzilla.gnome.org/show_bug.cgi?id=660165
gst: Fix compiler warnings on 64 bit mingw-w64
Fixes bug #660083.
Fixes bug #660083.
examples: fix bogus g_object_unref in helloworld example
GMainLoop is not a GObject.
https://bugzilla.gnome.org/show_bug.cgi?id=424143
GMainLoop is not a GObject.
https://bugzilla.gnome.org/show_bug.cgi?id=424143
basetransform: delay serialized events when src caps are not set yet
https://bugzilla.gnome.org/show_bug.cgi?id=659571
https://bugzilla.gnome.org/show_bug.cgi?id=659571
pad: Set caps on pad before checking if the pad is linked
This allows the setcaps handler and notify::caps to link
the pad downstream and doesn't require hacks to always
provide a peer to the pad, like in decodebin2.
This allows the setcaps handler and notify::caps to link
the pad downstream and doesn't require hacks to always
provide a peer to the pad, like in decodebin2.
12 years agocaps: use g_value_take_string() and gst_value_get_caps() instead of accessing interna...
caps: use g_value_take_string() and gst_value_get_caps() instead of accessing internal fields
pad: add more debug logging for other chain function code path as well
pad: fix up printf format in debug message
Which I messed up.
Which I messed up.
pad: make some debug traces more useful
https://bugzilla.gnome.org/show_bug.cgi?id=659139
https://bugzilla.gnome.org/show_bug.cgi?id=659139
scripts: remove gstcvstest.sh
lfocontrolsource: fix clang compiler warning
Cast enum to int before checking for negative values, which are
impossible according to the enum list.
gstlfocontrolsource.c:652:45: error: comparison of unsigned enum expression < 0
is always false [-Werror,-Wtautological-compare]
if (waveform >= num_waveforms || waveform < 0) {
~~~~~~~~ ^ ~
https://bugzilla.gnome.org/show_bug.cgi?id=653137
Cast enum to int before checking for negative values, which are
impossible according to the enum list.
gstlfocontrolsource.c:652:45: error: comparison of unsigned enum expression < 0
is always false [-Werror,-Wtautological-compare]
if (waveform >= num_waveforms || waveform < 0) {
~~~~~~~~ ^ ~
https://bugzilla.gnome.org/show_bug.cgi?id=653137
tests: make sure filesrc returns escaped URIs even if the input was unescaped
https://bugzilla.gnome.org/show_bug.cgi?id=654673
https://bugzilla.gnome.org/show_bug.cgi?id=654673
caps: move log messages for caps creation/freeing into TRACE category
Reduce SPAM for GST_CAPS:5.
Reduce SPAM for GST_CAPS:5.
pad: Only do the subset check in gst_pad_accept_caps() if the pad claims to accept the caps
basesink: make it easy to override the pad query
Add a vmethod to handle the pad query.
Install a default handler for the pad query.
Use the new query function in filesink
Add a vmethod to handle the pad query.
Install a default handler for the pad query.
Use the new query function in filesink
Revert "pad: Use gst_pad_accept_caps() instead of manually checking when configuring a sinkpad"
This reverts commit d3cad28da936b037d877dc70c02286b81b680284.
It causes performance problems because acceptcaps() propagates downstream.
This reverts commit d3cad28da936b037d877dc70c02286b81b680284.
It causes performance problems because acceptcaps() propagates downstream.
pad: Use gst_pad_accept_caps() instead of manually checking when configuring a sinkpad
pad: Print a g_warning() if pad accept caps that are not a subset of its caps
In 0.11 only subsets are supported again as documented instead of also
allowing non-empty intersections.
In 0.11 only subsets are supported again as documented instead of also
allowing non-empty intersections.
Revert "basetransform: Use check for subsets and not non-empty intersection to check if caps are compatible"
This reverts commit 5e5cc5e89e7e2858a6352fa4c81a374f6e5a6297.
See bug #658541.
This reverts commit 5e5cc5e89e7e2858a6352fa4c81a374f6e5a6297.
See bug #658541.
Revert "pad: Check for subsets, not non-empty intersections to check if caps are compatible"
This reverts commit 0c5d50207326d74a4805bcd898bfac887540f12b.
See bug #658541.
This reverts commit 0c5d50207326d74a4805bcd898bfac887540f12b.
See bug #658541.
docs, gst: typo fixes
https://bugzilla.gnome.org/show_bug.cgi?id=658449
https://bugzilla.gnome.org/show_bug.cgi?id=658449
docs: fix make distcheck
No point removin those empty override files from git, they'll
just be re-created later, so let's tell gtk-doc about them, so
it can clean them up properly.
No point removin those empty override files from git, they'll
just be re-created later, so let's tell gtk-doc about them, so
it can clean them up properly.
basetransform: If there's no peer we still have to transform ANY caps in getcaps()
Otherwise elements like capsfilter will return ANY caps if no
peer is present instead of the filter caps. The transform_caps()
vfunc could do transformations to the template caps that do not
result in the unmodified template caps.
Otherwise elements like capsfilter will return ANY caps if no
peer is present instead of the filter caps. The transform_caps()
vfunc could do transformations to the template caps that do not
result in the unmodified template caps.
docs: cleanup makefiles
Remove commented out parts that we don't need. Remove "the wingo addition" - no
so useful after all. Narrow down file-globs for plugin docs.
Remove commented out parts that we don't need. Remove "the wingo addition" - no
so useful after all. Narrow down file-globs for plugin docs.
docs: escape % in docblob
ghostpad: Use gst_pad_set_caps() instead of manually changing caps
gst_pad_set_caps() does essentially the same but additionally calls
the pad's setcaps function.
Fixes bug #658076.
gst_pad_set_caps() does essentially the same but additionally calls
the pad's setcaps function.
Fixes bug #658076.
basetransform: Use check for subsets and not non-empty intersection to check if caps are compatible
pad: Check for subsets, not non-empty intersections to check if caps are compatible
Pads should only accept caps that are a subset of the pad caps, e.g.
they should accept only caps that have a non-empty intersection and
at least all fields of the pad caps.
Without this a pad that wants for example
"video/x-h264,stream-format=byte-stream"
will be happy to accept
"video/x-h264".
Pads should only accept caps that are a subset of the pad caps, e.g.
they should accept only caps that have a non-empty intersection and
at least all fields of the pad caps.
Without this a pad that wants for example
"video/x-h264,stream-format=byte-stream"
will be happy to accept
"video/x-h264".
docs: add since 0.10.36 on the new _NONE enum values
stepping: use the proper argument order
Fixes warning #188: enumerated type mixed with another type reported by ICC.
Fixes warning #188: enumerated type mixed with another type reported by ICC.
fileindex: explicitly cast to the enum types
Fixes warning #188: enumerated type mixed with another type reported by ICC.
Fixes warning #188: enumerated type mixed with another type reported by ICC.
typefinder: use GST_TYPE_FIND_NONE instead of 0
Fixes warning #188: enumerated type mixed with another type reported by ICC.
Fixes warning #188: enumerated type mixed with another type reported by ICC.
lfocontrolsource: explicitly cast to the enum type
Fixes warning #188: enumerated type mixed with another type reported by ICC.
Fixes warning #188: enumerated type mixed with another type reported by ICC.
typefind: add GST_TYPE_FIND_NONE and use it
Fixes warning #188: enumerated type mixed with another type reported by ICC.
Fixes warning #188: enumerated type mixed with another type reported by ICC.