platform/upstream/gstreamer.git
13 years agobaseaudiocodec: ... and also rename to baseaudiodecoder
Mark Nauwelaerts [Tue, 1 Mar 2011 13:08:18 +0000 (14:08 +0100)]
baseaudiocodec: ... and also rename to baseaudiodecoder

13 years agogst-libs/gst/audio: Remove baseaudiodecoder
Mark Nauwelaerts [Tue, 1 Mar 2011 12:58:31 +0000 (13:58 +0100)]
gst-libs/gst/audio: Remove baseaudiodecoder

Adds little beyond baseaudiocodec (seeking, bit of query), and what it adds
is mainly out-of-scope (e.g. decoder seeking, should be done by upstream
demuxer/parser) and/or based on non-prime example (mad).

13 years agobaseaudiodecoder: Return TRUE if we run into special conversion cases.
Iago Toral [Thu, 17 Sep 2009 11:26:28 +0000 (13:26 +0200)]
baseaudiodecoder: Return TRUE if we run into special conversion cases.

13 years agoaudio: initial version of GstBaseAudioCodec
Iago Toral [Tue, 1 Sep 2009 12:17:53 +0000 (14:17 +0200)]
audio: initial version of GstBaseAudioCodec

Moved most of the code to GstBaseAudioCodec, GstBaseAudioDecode is
now really small, maybe we do not really need it (or its encoder
counterpart). Added more API for subclasses and documentation.

13 years agoAdded src_queries to decoder class. Added handle_discont to decoder
Iago Toral [Fri, 14 Aug 2009 07:45:52 +0000 (09:45 +0200)]
Added src_queries to decoder class. Added handle_discont to decoder
class. Reworked reset. Various other minor fixes.

13 years agoAdded a draft implementation of gstbaseaudiodecoder
Iago Toral [Thu, 6 Aug 2009 13:28:00 +0000 (15:28 +0200)]
Added a draft implementation of gstbaseaudiodecoder

13 years agoAdded audio directory for audio codec base classes
Mark Nauwelaerts [Tue, 1 Mar 2011 10:56:29 +0000 (11:56 +0100)]
Added audio directory for audio codec base classes

13 years agoaudioencoders: add streamheader helper utility
Mark Nauwelaerts [Fri, 18 Feb 2011 15:38:37 +0000 (16:38 +0100)]
audioencoders: add streamheader helper utility

13 years agoaudioencoders: baseaudioencoder and ported encoders
Mark Nauwelaerts [Thu, 27 Jan 2011 15:52:50 +0000 (16:52 +0100)]
audioencoders: baseaudioencoder and ported encoders

13 years agowin32: Add new discoverer API
Sebastian Dröge [Fri, 26 Aug 2011 08:03:26 +0000 (10:03 +0200)]
win32: Add new discoverer API

13 years agodocs: Add new discoverer API
Sebastian Dröge [Fri, 26 Aug 2011 08:03:17 +0000 (10:03 +0200)]
docs: Add new discoverer API

13 years agodiscoverer: retrieve audio track language from tags too
Vincent Penquerc'h [Wed, 24 Aug 2011 15:29:08 +0000 (16:29 +0100)]
discoverer: retrieve audio track language from tags too

https://bugzilla.gnome.org/show_bug.cgi?id=657257

13 years agodiscoverer: consider subtitles as raw
Vincent Penquerc'h [Wed, 24 Aug 2011 14:09:47 +0000 (15:09 +0100)]
discoverer: consider subtitles as raw

Otherwise, discoverer will generated an "inner" codec
where there can be a tranformation (eg, kate -> DVD SPU,
and various ->text/x-pango-markup).

https://bugzilla.gnome.org/show_bug.cgi?id=639055

13 years agodiscoverer: add application/x-kate to subtitles caps
Vincent Penquerc'h [Wed, 24 Aug 2011 14:05:38 +0000 (15:05 +0100)]
discoverer: add application/x-kate to subtitles caps

https://bugzilla.gnome.org/show_bug.cgi?id=639055

13 years agodiscoverer: get language from other tags if we did not get it already
Vincent Penquerc'h [Wed, 24 Aug 2011 13:59:38 +0000 (14:59 +0100)]
discoverer: get language from other tags if we did not get it already

https://bugzilla.gnome.org/show_bug.cgi?id=639055

13 years agodiscoverer: add subtitles API
Vincent Penquerc'h [Wed, 24 Aug 2011 14:04:50 +0000 (15:04 +0100)]
discoverer: add subtitles API

https://bugzilla.gnome.org/show_bug.cgi?id=639055

13 years agoplayback: reference count ts_offset
David Schleef [Sun, 21 Aug 2011 21:51:45 +0000 (14:51 -0700)]
playback: reference count ts_offset

Apparently this object is being used after it's freed.  This is one
way to fix it, although perhaps not the best way.  Fixes: #656715.

13 years agotheoraenc: fix caps leak
Vincent Penquerc'h [Thu, 25 Aug 2011 13:55:14 +0000 (14:55 +0100)]
theoraenc: fix caps leak

https://bugzilla.gnome.org/show_bug.cgi?id=657333

13 years agobasertppayload: Make perfect timestamps reproducible across element restart
Olivier Crête [Sat, 9 Jul 2011 03:06:46 +0000 (23:06 -0400)]
basertppayload: Make perfect timestamps reproducible across element restart

Without the perfect timestamp machinery, the RTP timestamp can be
computed directly from the running time of a buffer, but the perfect
timestamp patch broke that assumption. This patch restores it by
having the first perfect timestamp be the running time of that buffer
and counting from there.

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=654434

13 years agooggmux: fix leaks in skeleton writing
Vincent Penquerc'h [Wed, 24 Aug 2011 16:39:11 +0000 (17:39 +0100)]
oggmux: fix leaks in skeleton writing

https://bugzilla.gnome.org/show_bug.cgi?id=563251

13 years agooggmux: generate message headers from received tags
Vincent Penquerc'h [Thu, 18 Aug 2011 15:36:23 +0000 (16:36 +0100)]
oggmux: generate message headers from received tags

Some message headers can be deduced from tags (eg, "Language").

https://bugzilla.gnome.org/show_bug.cgi?id=563251

13 years agoogg: use memory slices where appropriate
Vincent Penquerc'h [Thu, 18 Aug 2011 09:05:17 +0000 (10:05 +0100)]
ogg: use memory slices where appropriate

While there, avoid zeroing newly allocated memory where unnecessary

https://bugzilla.gnome.org/show_bug.cgi?id=656775

13 years agoplaysink{audio,video}convert: Send NEWSEGMENT events to sinkpads instead of pushing...
Sebastian Dröge [Wed, 24 Aug 2011 12:05:27 +0000 (14:05 +0200)]
playsink{audio,video}convert: Send NEWSEGMENT events to sinkpads instead of pushing them

13 years agooggdemux: do not warn when reaching EOS while scanning for the end chain
Vincent Penquerc'h [Tue, 23 Aug 2011 10:12:10 +0000 (11:12 +0100)]
oggdemux: do not warn when reaching EOS while scanning for the end chain

After all, we were asking for it.

This gets rid of the last warning-about-expected-condition.

w00t.

https://bugzilla.gnome.org/show_bug.cgi?id=657151

13 years agooggdemux: add media type to chain information reports
Vincent Penquerc'h [Tue, 23 Aug 2011 10:08:25 +0000 (11:08 +0100)]
oggdemux: add media type to chain information reports

One more little step in making logs a little less abstruse.

https://bugzilla.gnome.org/show_bug.cgi?id=657151

13 years agooggstream: correctly identify skeleton EOS packet
Vincent Penquerc'h [Tue, 23 Aug 2011 10:05:11 +0000 (11:05 +0100)]
oggstream: correctly identify skeleton EOS packet

It is 0 byte, and was triggering the "bad packet" logic.

https://bugzilla.gnome.org/show_bug.cgi?id=657151

13 years agooggdemux: do not warn about expected occurences
Vincent Penquerc'h [Tue, 23 Aug 2011 09:58:20 +0000 (10:58 +0100)]
oggdemux: do not warn about expected occurences

In this case, finding a skeleton packet.
Once upon a time, it used to be rare indeed, but no more.

https://bugzilla.gnome.org/show_bug.cgi?id=657151

13 years agooggdemux: do not warn when finding a non BOS page
Vincent Penquerc'h [Tue, 23 Aug 2011 09:47:53 +0000 (10:47 +0100)]
oggdemux: do not warn when finding a non BOS page

After all, we do hope to find actual data for these streams.
However, warn if we could not set up a chain when we find a
non BOS page, as that means we don't have a valid Ogg stream.

https://bugzilla.gnome.org/show_bug.cgi?id=657151

13 years agooggdemux: rename local variable for clarity
Vincent Penquerc'h [Tue, 23 Aug 2011 09:40:12 +0000 (10:40 +0100)]
oggdemux: rename local variable for clarity

While the casual reader might end up bewildered by just why this
change might increase clarity, it just happens than, in the libogg
and associated sources, op is the canonical name for an ogg_packet
whlie og is the canonical name for an ogg_page, and reading this
code confuses me.

https://bugzilla.gnome.org/show_bug.cgi?id=657151

13 years agooggdemux: do not try to determine duration of header packets
Vincent Penquerc'h [Tue, 23 Aug 2011 09:32:36 +0000 (10:32 +0100)]
oggdemux: do not try to determine duration of header packets

Headers are inherently durationless.
Instead, set duration to 0 to avoid increasing tracked granpos,
and do not warn about it, since it is totally expected.

https://bugzilla.gnome.org/show_bug.cgi?id=657151

13 years agooggstream: include stream type in warnings
Vincent Penquerc'h [Tue, 23 Aug 2011 09:29:49 +0000 (10:29 +0100)]
oggstream: include stream type in warnings

It makes it easier to work out what's going on.

https://bugzilla.gnome.org/show_bug.cgi?id=657151

13 years agooggstream: set skeleton stream media type to application/x-ogg-skeleton
Vincent Penquerc'h [Tue, 23 Aug 2011 09:28:33 +0000 (10:28 +0100)]
oggstream: set skeleton stream media type to application/x-ogg-skeleton

This is to match the typefinder, and to make logs clearer.

https://bugzilla.gnome.org/show_bug.cgi?id=657151

13 years agooggmux: add skeleton write support
Vincent Penquerc'h [Wed, 17 Aug 2011 16:09:44 +0000 (17:09 +0100)]
oggmux: add skeleton write support

Version written is 3.0

Base times are left empty for now.

Content-Type should be the MIME type of the stream. It is set to
the GStreamer media type for now, which is probably the same for
the streams oggmux supports.

https://bugzilla.gnome.org/show_bug.cgi?id=563251

13 years agooggdemux: do not skip sparse streams when determining start times
Vincent Penquerc'h [Mon, 22 Aug 2011 13:56:38 +0000 (14:56 +0100)]
oggdemux: do not skip sparse streams when determining start times

This fixes demuxing of streams containing only sparse streams,
which would cause an infinite loop in _read_end_chain.

https://bugzilla.gnome.org/show_bug.cgi?id=657062

13 years agooggdemux: do not ignore sparse streams' start time
Vincent Penquerc'h [Mon, 22 Aug 2011 13:55:59 +0000 (14:55 +0100)]
oggdemux: do not ignore sparse streams' start time

But do not wait for them either, if we don't have a packet for them.

https://bugzilla.gnome.org/show_bug.cgi?id=657062

13 years agovorbisenc: Relax overly-tight jitter tolerances in gstvobisenc
Monty Montgomery [Thu, 21 Jul 2011 21:16:26 +0000 (17:16 -0400)]
vorbisenc: Relax overly-tight jitter tolerances in gstvobisenc

vorbisenc currently reacts in a rater draconian fashion if input
timestamps are more than 1/2 sample off what it considers ideal. If data
is 'too late' it truncates buffers, if it is 'too soon' it completely
shuts down encode and restarts it.  This is causingvorbisenc to produce
corrupt output when encoding data produced by sources with bugs that
produce a smple or two of jitter (eg, flacdec)

13 years agotextoverlay: fix text buffer leak
Vincent Penquerc'h [Mon, 22 Aug 2011 08:06:53 +0000 (09:06 +0100)]
textoverlay: fix text buffer leak

Make sure to always unref the input text buffer.

Reported by bcxa.sz@gmail.com.

https://bugzilla.gnome.org/show_bug.cgi?id=657049

13 years agodocs: fix xref for the property
Stefan Kost [Sat, 20 Aug 2011 17:46:31 +0000 (19:46 +0200)]
docs: fix xref for the property

13 years agodocs: handle warnings emitted by gtk-doc
Stefan Kost [Sat, 20 Aug 2011 17:16:42 +0000 (19:16 +0200)]
docs: handle warnings emitted by gtk-doc

This is useful and in most cases someone had put arbitrary markup into the docs,
misspelled xref'ed symbols, forgot to add stuff to the docs etc..

13 years agodocs: partially revert my last commit
Stefan Kost [Sat, 20 Aug 2011 15:53:11 +0000 (17:53 +0200)]
docs: partially revert my last commit

Somehow this was already there, but I missed that commit.

13 years agodocs: add new taglicense docs and clean them up
Stefan Kost [Sat, 20 Aug 2011 12:11:11 +0000 (14:11 +0200)]
docs: add new taglicense docs and clean them up

Avoid ugly docbook tags unless needed.

13 years agopo: update for new translatable string
Tim-Philipp Müller [Sat, 20 Aug 2011 11:37:10 +0000 (12:37 +0100)]
po: update for new translatable string

13 years agotag: fix distcheck issue
Tim-Philipp Müller [Sat, 20 Aug 2011 11:36:20 +0000 (12:36 +0100)]
tag: fix distcheck issue

Dist licenses dict.

13 years agoogg: do not use 32 bit modifiers to print serial numbers
Vincent Penquerc'h [Thu, 18 Aug 2011 15:20:57 +0000 (16:20 +0100)]
ogg: do not use 32 bit modifiers to print serial numbers

If ints are 64 bits, 32 bits should get promoted in varargs anyway,
and we don't care about 16 bit ints.
This makes the code a lot more readable, and still gets us nice
hexadecimal 32 bit serialnos.

https://bugzilla.gnome.org/show_bug.cgi?id=656775

13 years agoplaysink: Reconfigure when pads are added later
Edward Hervey [Wed, 27 Jul 2011 11:05:31 +0000 (11:05 +0000)]
playsink: Reconfigure when pads are added later

Instead of just assuming all pads are created at the same time,
remember which ones are actually new (via ->pending_blocked_pads).

This allows the following use-case to properly work:
* Upstream starts with audio-only
* Only that pad gets data, blocks and a real audio sink is created
* Upstream laters adds a video stream
* A new pad is requested, blocks and reconfiguration kicks in in
  order to add a new real video sink

13 years agoogg: get the operator precedence right, even if only a doc
Vincent Penquerc'h [Thu, 18 Aug 2011 08:37:38 +0000 (09:37 +0100)]
ogg: get the operator precedence right, even if only a doc

https://bugzilla.gnome.org/show_bug.cgi?id=656775

13 years agooggstream: vorbis has a preroll of 2
Vincent Penquerc'h [Thu, 18 Aug 2011 08:30:46 +0000 (09:30 +0100)]
oggstream: vorbis has a preroll of 2

https://bugzilla.gnome.org/show_bug.cgi?id=656775

13 years agooggstream: new convenience function to get a stream's media type
Vincent Penquerc'h [Wed, 17 Aug 2011 18:40:08 +0000 (19:40 +0100)]
oggstream: new convenience function to get a stream's media type

This will make logging a lot clearer, both in code and in output.

https://bugzilla.gnome.org/show_bug.cgi?id=656775

13 years agoogg: move the "always flush page" to oggstream
Vincent Penquerc'h [Wed, 17 Aug 2011 17:48:54 +0000 (18:48 +0100)]
ogg: move the "always flush page" to oggstream

It avoids checking for specific media types in the muxer.

https://bugzilla.gnome.org/show_bug.cgi?id=656775

13 years agooggmux: use oggstream to decide which BOS packets to place first
Vincent Penquerc'h [Wed, 17 Aug 2011 17:38:39 +0000 (18:38 +0100)]
oggmux: use oggstream to decide which BOS packets to place first

Ogg recommends video BOS packets to be first.
Use the "is_video" flag in oggstream to select those, rather than
check for known mime types.

https://bugzilla.gnome.org/show_bug.cgi?id=656775

13 years agoogg: rationalize serialno type to guint32
Vincent Penquerc'h [Wed, 17 Aug 2011 17:03:16 +0000 (18:03 +0100)]
ogg: rationalize serialno type to guint32

It is a 32 bit unsigned number.
Sure, the libogg API uses a long, but that's an unfortunate oversight.

https://bugzilla.gnome.org/show_bug.cgi?id=656775

13 years agooggmux: factor the header packet creation code
Vincent Penquerc'h [Wed, 17 Aug 2011 16:39:18 +0000 (17:39 +0100)]
oggmux: factor the header packet creation code

https://bugzilla.gnome.org/show_bug.cgi?id=656775

13 years agooggmux: headers should always have granpos 0
Vincent Penquerc'h [Wed, 17 Aug 2011 16:18:47 +0000 (17:18 +0100)]
oggmux: headers should always have granpos 0

https://bugzilla.gnome.org/show_bug.cgi?id=656775

13 years agoaudioresample: fix build without orc
Vincent Penquerc'h [Thu, 18 Aug 2011 08:48:16 +0000 (09:48 +0100)]
audioresample: fix build without orc

https://bugzilla.gnome.org/show_bug.cgi?id=656781

13 years agotag: id3: avoid some more relocations in genre table
Tim-Philipp Müller [Mon, 15 Aug 2011 00:22:02 +0000 (01:22 +0100)]
tag: id3: avoid some more relocations in genre table

13 years agoaudioresample: add FFT based checks
Vincent Penquerc'h [Fri, 12 Aug 2011 11:07:32 +0000 (12:07 +0100)]
audioresample: add FFT based checks

Send a few simple tones through audioresample and check
that the main frequency spot is the same for the input and
the resampled output.

https://bugzilla.gnome.org/show_bug.cgi?id=656392

13 years agortspconnection: add OSX specific hack to detect when a connection is refused
Alessandro Decina [Mon, 15 Aug 2011 21:41:24 +0000 (23:41 +0200)]
rtspconnection: add OSX specific hack to detect when a connection is refused

Unlike linux, OSX wakes up select with POLLOUT (instead of POLLERR) when
connect() is done async and the connection is refused. Therefore always check
for the socket error state using getsockopt (..., SO_ERROR, ...) after a
connection attempt.

13 years agodocs: add new license API to docs
Tim-Philipp Müller [Sun, 14 Aug 2011 23:17:14 +0000 (00:17 +0100)]
docs: add new license API to docs

13 years agoconfigure: try pkg-config first when looking for zlib
Tim-Philipp Müller [Sun, 14 Aug 2011 23:03:39 +0000 (00:03 +0100)]
configure: try pkg-config first when looking for zlib

13 years agotag: id3v2: add specs to git for reference
Tim-Philipp Müller [Sun, 14 Aug 2011 19:44:19 +0000 (20:44 +0100)]
tag: id3v2: add specs to git for reference

13 years agotag: id3v2: avoid some relocations, make table static
Tim-Philipp Müller [Sun, 14 Aug 2011 12:32:12 +0000 (13:32 +0100)]
tag: id3v2: avoid some relocations, make table static

13 years agotag: id3v2: add debug category for ID3 tag parsing
Tim-Philipp Müller [Sun, 14 Aug 2011 00:47:41 +0000 (01:47 +0100)]
tag: id3v2: add debug category for ID3 tag parsing

13 years agotag: id3v2: add id3v2 tag parsing helpers
Mark Nauwelaerts [Mon, 18 Jul 2011 16:09:53 +0000 (18:09 +0200)]
tag: id3v2: add id3v2 tag parsing helpers

https://bugzilla.gnome.org/show_bug.cgi?id=654388

13 years agotag: id3v2: return ID3TAGS_BROKEN_TAG for unsupported versions
Stefan Kost [Tue, 22 Feb 2011 13:19:00 +0000 (15:19 +0200)]
tag: id3v2: return ID3TAGS_BROKEN_TAG for unsupported versions

This prevents us for trying to work with a NULL taglist.

13 years agotag: id3v2: fix parsing of ID3v2.4 genre frames with multiple genres
Erich Schubert [Sun, 2 Jan 2011 19:23:51 +0000 (19:23 +0000)]
tag: id3v2: fix parsing of ID3v2.4 genre frames with multiple genres

We'd only extract the first genre (multiple times) instead of all
genres.

https://bugzilla.gnome.org/show_bug.cgi?id=638535

13 years agotag: id3v2: Sanitize id3 frame names
Edward Hervey [Fri, 24 Sep 2010 13:19:15 +0000 (15:19 +0200)]
tag: id3v2: Sanitize id3 frame names

This is similar to what is done in qtdemux. Avoids providing invalid
structure/tags names

13 years agotag: id3v2: fix parsing of unsynced frames with data length indicator
Tim-Philipp Müller [Tue, 30 Mar 2010 00:50:32 +0000 (01:50 +0100)]
tag: id3v2: fix parsing of unsynced frames with data length indicator

Fixes bug #614158.

13 years agoAdd -Wwrite-strings to the configure flags
Benjamin Otte [Fri, 19 Mar 2010 23:54:14 +0000 (00:54 +0100)]
Add -Wwrite-strings to the configure flags

... and fix all warnings

13 years agotag: id3v2: prefer two letter ISO 639-1 code for extended comment
Tim-Philipp Müller [Sun, 13 Dec 2009 13:19:43 +0000 (13:19 +0000)]
tag: id3v2: prefer two letter ISO 639-1 code for extended comment

13 years agotag: id3v2: fixes warnings building on macosx
Josep Torra [Fri, 9 Oct 2009 13:59:25 +0000 (15:59 +0200)]
tag: id3v2: fixes warnings building on macosx

Another round on the formating of that debug line.

13 years agotag: id3v2: cast pointer math results to glong
Stefan Kost [Fri, 9 Oct 2009 11:44:02 +0000 (14:44 +0300)]
tag: id3v2: cast pointer math results to glong

13 years agotag: id3v2: don't cast, but use the right format specified instead
Stefan Kost [Fri, 9 Oct 2009 10:38:17 +0000 (13:38 +0300)]
tag: id3v2: don't cast, but use the right format specified instead

This correct some of the previous macos fixes.

13 years agotag: id3v2: fix printf warnings on macosx
Josep Torra [Fri, 9 Oct 2009 09:42:36 +0000 (11:42 +0200)]
tag: id3v2: fix printf warnings on macosx

13 years agotag: id3v2: fprintf, sprintf, sscanf need stdio.h
Stefan Kost [Wed, 7 Oct 2009 11:03:20 +0000 (14:03 +0300)]
tag: id3v2: fprintf, sprintf, sscanf need stdio.h

13 years agotag: id3v2: Fix compile warnings with gcc 4.0.1.
Alessandro Decina [Tue, 22 Sep 2009 13:03:20 +0000 (15:03 +0200)]
tag: id3v2: Fix compile warnings with gcc 4.0.1.

13 years agotag: id3v2: Try GST_*_TAG_ENCODING and locale encoding if tags are not UTF8
LoneStar [Sun, 9 Aug 2009 10:52:17 +0000 (12:52 +0200)]
tag: id3v2: Try GST_*_TAG_ENCODING and locale encoding if tags are not UTF8

Fixes bug #499242.

13 years agotag: id3v2: sizes in ID3 v2.3 are unlikely to be sync-safe integers
Tim-Philipp Müller [Fri, 7 Aug 2009 15:42:39 +0000 (16:42 +0100)]
tag: id3v2: sizes in ID3 v2.3 are unlikely to be sync-safe integers

In ID3 v2.3 compressed frames will have a 4-byte data length indicator
after the frame header to indicate the size of the decompressed data.
This integer is unlikely to be a sync-safe integer for v2.3 tags,
only in v2.4 it's sync-safe.

13 years agotag: id3v2: fix typo in debug message
Tim-Philipp Müller [Fri, 7 Aug 2009 15:36:55 +0000 (16:36 +0100)]
tag: id3v2: fix typo in debug message

13 years agotag: id3v2: fix parsing of unsync'ed ID3 v2.4 tags and frames
Tim-Philipp Müller [Fri, 7 Aug 2009 15:02:23 +0000 (16:02 +0100)]
tag: id3v2: fix parsing of unsync'ed ID3 v2.4 tags and frames

Reversing the unsynchronisation seems to work slightly differently
for ID3 v2.3 tags and v2.4 tags: v2.3 tags don't have syncsafe frame
sizes in the frame header, so the unsynchronisation is applied to
the whole frame data including all the frame headers. v2.4 frames
have sync-safe sizes, however, so the unsynchronisation only needs
to be applied to the actual frame data, and it seems that's what's
being done as well. So we need to undo the unsynchronisation on a
per-frame basis for v2.4 tags for things to work properly.

Fixes extraction of coverart/images from APIC frames in ID3 v2.4
tags (#588148).

Add unit test for this as well.

13 years agotag: id3v2: parse unsynchronised tags properly
Tim-Philipp Müller [Fri, 24 Apr 2009 00:51:35 +0000 (01:51 +0100)]
tag: id3v2: parse unsynchronised tags properly

We didn't handle unsynchronization at all up to now, which might have
caused frames to not be extracted - esp. frames after an APIC picture
frame. Fixes #577468.

13 years agotag: id3v2: pass the right size value for size of all frames to the parser
Tim-Philipp Müller [Fri, 24 Apr 2009 00:01:53 +0000 (01:01 +0100)]
tag: id3v2: pass the right size value for size of all frames to the parser

Frame data size is tag size adjusted for size of the tag header and
footer, not tag size including header and footer.

13 years agotag: id3v2: Use new utility functions in libgsttag to process coverart (#512333).
Tim-Philipp Müller [Wed, 4 Jun 2008 10:42:46 +0000 (10:42 +0000)]
tag: id3v2: Use new utility functions in libgsttag to process coverart (#512333).

Original commit message from CVS:
* ext/flac/gstflacdec.c: (gst_flac_extract_picture_buffer):
* gst-libs/gst/tag/id3v2frames.c: (parse_picture_frame):
Use new utility functions in libgsttag to process coverart (#512333).

13 years agotag: id3v2: Generate the image-type values correctly. Leave them out of the caps...
Jan Schmidt [Fri, 11 Jan 2008 21:08:59 +0000 (21:08 +0000)]
tag: id3v2: Generate the image-type values correctly. Leave them out of the caps when outputting a "preview image" tag, since it ...

Original commit message from CVS:
* ext/flac/gstflacdec.c: (gst_flac_extract_picture_buffer):
* gst-libs/gst/tag/id3v2frames.c: (parse_picture_frame):
Generate the image-type values correctly. Leave them out of the caps
when outputting a "preview image" tag, since it only makes sense
to have one of those - the type is irrelevant.
* sys/sunaudio/gstsunaudiomixerctrl.c:
(gst_sunaudiomixer_ctrl_open):
If we can, mark the mixer multiple open when we use it, in case
(for some reason) the process wants to open it again elsewhere.

13 years agotag: id3v2: Make sure the ISO 639-X language code in ID3v2 COMM frames so we don...
Tommi Myöhänen [Wed, 9 Jan 2008 15:20:19 +0000 (15:20 +0000)]
tag: id3v2: Make sure the ISO 639-X language code in ID3v2 COMM frames so we don't end up with non-UT...

Original commit message from CVS:
Based on patch by: Tommi Myöhänen <ext-tommi.myohanen nokia com>
* gst-libs/gst/tag/id3v2frames.c: (parse_comment_frame):
Make sure the ISO 639-X language code in ID3v2 COMM frames
is actually valid UTF-8 (or rather: ASCII), so we don't end
up with non-UTF8 strings in tags if there's garbage in the
language field. Also make sure the language code is always
lower case. Fixes: #508291.

13 years agotag: id3v2: Parse WOAF frames and put the result into GST_TAG_CONTACT, which is where...
Tim-Philipp Müller [Fri, 14 Dec 2007 10:17:10 +0000 (10:17 +0000)]
tag: id3v2: Parse WOAF frames and put the result into GST_TAG_CONTACT, which is where it would end up...

Original commit message from CVS:
* tag: id3v2: (parse_url_link_frame):
Parse WOAF frames and put the result into GST_TAG_CONTACT,
which is where it would end up if the same information was
put in a vorbis comment (don't think it's worth adding a
new URI tag for this). Fixes #488112.

13 years agotag: id3v2: We don't want the same string multiple times in a tag list for the same...
Tim-Philipp Müller [Wed, 14 Nov 2007 21:39:47 +0000 (21:39 +0000)]
tag: id3v2: We don't want the same string multiple times in a tag list for the same tag ever, for any tag, not jus...

Original commit message from CVS:
* gst-libs/gst/tag/id3v2.c:
* gst-libs/gst/tag/id3v2.h:
* gst-libs/gst/tag/id3v2frames.c: (id3v2_tag_to_taglist):
We don't want the same string multiple times in a tag list for the
same tag ever, for any tag, not just for GST_TAG_GENRE, so make sure
this doesn't happen and remove special-case code for GST_TAG_GENRE.

13 years agotag: id3v2: Extract license/copyright URIs from ID3v2 WCOP frames (Fixes #447000).
Jason Kivlighn [Thu, 11 Oct 2007 17:55:29 +0000 (17:55 +0000)]
tag: id3v2: Extract license/copyright URIs from ID3v2 WCOP frames (Fixes #447000).

Original commit message from CVS:
Based on patch by: Jason Kivlighn  <jkivlighn gmail com>
* gst-libs/gst/tag/id3v2frames.c:
Extract license/copyright URIs from ID3v2 WCOP frames
(Fixes #447000).
* tests/check/elements/id3demux.c:
* tests/files/Makefile.am:
* tests/files/id3-447000-wcop.tag:
Add simple unit test.

13 years agotag: id3v2: Port ID3 tag demuxer over to the new GstTagDemux in -base (now would...
Tim-Philipp Müller [Sat, 6 Oct 2007 16:13:14 +0000 (16:13 +0000)]
tag: id3v2: Port ID3 tag demuxer over to the new GstTagDemux in -base (now would be a good time to test re-importi...

Original commit message from CVS:
* gst-libs/gst/tag/gstid3demux.c:
* gst-libs/gst/tag/gstid3demux.h:
* gst-libs/gst/tag/id3v2.c:
* gst-libs/gst/tag/id3v2.h:
* gst-libs/gst/tag/id3v2frames.c:
Port ID3 tag demuxer over to the new GstTagDemux in -base
(now would be a good time to test re-importing your music
collection).

13 years agotag: id3v2: Fix parsing of ID3 v2.2.0 PIC frames. Only in version >= 2.3.0 is the...
Tim-Philipp Müller [Mon, 12 Mar 2007 13:28:29 +0000 (13:28 +0000)]
tag: id3v2: Fix parsing of ID3 v2.2.0 PIC frames. Only in version >= 2.3.0 is the image format a vari...

Original commit message from CVS:
* gst-libs/gst/tag/id3v2frames.c: (parse_picture_frame):
Fix parsing of ID3 v2.2.0 PIC frames. Only in version >= 2.3.0 is
the image format a variable-length NUL-terminated string; in
versions before that the image format is a fixed-length string of
3 characters (see #348644 for a sample tag).
Also make supplied mime type lower-case and fix up 'jpg' to 'jpeg'.

13 years agotag: id3v2: Do not convert obsolete TDA/TDAT frames to TDRC frames, otherwise the...
Tim-Philipp Müller [Tue, 6 Mar 2007 18:16:49 +0000 (18:16 +0000)]
tag: id3v2: Do not convert obsolete TDA/TDAT frames to TDRC frames, otherwise the four-digit number will be interp...

Original commit message from CVS:
* gst-libs/gst/tag/id3v2.c: (id3demux_id3v2_frames_to_tag_list):
* gst-libs/gst/tag/id3v2.h:
* gst-libs/gst/tag/id3v2frames.c: (id3demux_id3v2_parse_frame),
(parse_obsolete_tdat_frame):
Do not convert obsolete TDA/TDAT frames to TDRC frames, otherwise
the four-digit number will be interpreted as a year, whereas it is
month and day in DDMM format. Instead, parse TDAT frames and fix up
the date in the GST_TAG_DATE tag later if we also extracted a year.
Fixes #407349.

13 years agotag: id3v2: Make sure that g_free always gets called on the same pointer that was...
René Stadler [Sun, 19 Nov 2006 13:41:53 +0000 (13:41 +0000)]
tag: id3v2: Make sure that g_free always gets called on the same pointer that was returned by g_mallo...

Original commit message from CVS:
* gst-libs/gst/tag/id3v2frames.c: (id3demux_id3v2_parse_frame):
Make sure that g_free always gets called on the same pointer that was
returned by g_malloc.  Fixes #376594.
Do not leak memory if decompressed size is wrong.
Remove unneeded check of return value of g_malloc.
Patch by: René Stadler <mail@renestadler.de>

13 years agotag: id3v2: We require a -base more recent than 0.10.9, so it's safe to use
Tim-Philipp Müller [Wed, 1 Nov 2006 13:59:49 +0000 (13:59 +0000)]
tag: id3v2: We require a -base more recent than 0.10.9, so it's safe to use

Original commit message from CVS:
* gst-libs/gst/tag/id3v2frames.c: (parse_picture_frame):
We require a -base more recent than 0.10.9, so it's safe to use
GST_TYPE_TAG_IMAGE_TYPE unconditionally now.
* ext/dv/gstdvdec.c: (gst_dvdec_sink_event):
* ext/jpeg/gstjpegdec.c: (gst_jpeg_dec_sink_event):
Use _newsegment_full() now that we depend on a recent enough core.
* gst/wavparse/gstwavparse.c:
Remove cruft that we don't need any longer now that we depend on
a recent enough -base.

13 years agotag: id3v2: Printf format fixes.
Tim-Philipp Müller [Thu, 5 Oct 2006 16:37:33 +0000 (16:37 +0000)]
tag: id3v2: Printf format fixes.

Original commit message from CVS:
* ext/cairo/gsttimeoverlay.c:
(gst_cairo_time_overlay_update_font_height):
* ext/gdk_pixbuf/pixbufscale.c: (gst_pixbufscale_transform_caps):
* ext/jpeg/gstjpegdec.c: (gst_jpeg_dec_parse_image_data):
* ext/jpeg/gstjpegenc.c: (gst_jpegenc_chain):
* ext/jpeg/gstsmokedec.c: (gst_smokedec_chain):
* ext/jpeg/gstsmokeenc.c: (gst_smokeenc_chain):
* ext/libpng/gstpngdec.c: (user_endrow_callback):
* gst/auparse/gstauparse.c: (gst_au_parse_parse_header):
* gst/avi/gstavidemux.c: (gst_avi_demux_parse_superindex),
(gst_avi_demux_parse_subindex), (gst_avi_demux_parse_stream),
(gst_avi_demux_stream_data):
* gst/cutter/gstcutter.c: (gst_cutter_chain):
* gst/debug/efence.c: (gst_efence_buffer_alloc),
(gst_fenced_buffer_copy):
* gst-libs/gst/tag/id3v2frames.c: (id3demux_id3v2_parse_frame):
* gst/matroska/matroska-demux.c: (gst_matroska_demux_add_stream):
* gst/matroska/matroska-mux.c: (gst_matroska_mux_start):
* gst/rtsp/gstrtspsrc.c: (gst_rtspsrc_send),
(gst_rtspsrc_handle_message):
* gst/wavparse/gstwavparse.c: (gst_wavparse_stream_headers):
* sys/ximage/ximageutil.c: (ximageutil_xcontext_get):
Printf format fixes.

13 years agotag: id3v2: If strings in text fields are marked ISO8859-1, but contain valid UTF...
Jan Schmidt [Tue, 22 Aug 2006 13:53:34 +0000 (13:53 +0000)]
tag: id3v2: If strings in text fields are marked ISO8859-1, but contain valid UTF-8 already, then han...

Original commit message from CVS:
* gst-libs/gst/tag/id3v2frames.c: (parse_text_identification_frame),
(parse_insert_string_field):
If strings in text fields are marked ISO8859-1, but contain
valid UTF-8 already, then handle them as UTF-8 and ignore
the encoding. (#351794)

13 years agoconfigure.ac: Require CVS of GStreamer core and -base (for
Tim-Philipp Müller [Wed, 16 Aug 2006 13:01:32 +0000 (13:01 +0000)]
configure.ac: Require CVS of GStreamer core and -base (for

Original commit message from CVS:
* configure.ac:
Require CVS of GStreamer core and -base (for
GST_TAG_EXTENDED_COMMENT and gst_tag_parse_extended_comment()).
* ext/taglib/gstid3v2mux.cc:
Write extended comment tags properly (#348762).
* gst-libs/gst/tag/id3v2frames.c: (id3demux_id3v2_parse_frame),
(parse_comment_frame):
Extract COMM frames into extended comments, which makes it
easier to properly retain the description bit of the tag
and maintain this information when re-tagging (#348762).

13 years agotag: id3v2: Extract frames for ID3v2 versions prior to ID3v2.3.0 properly as well...
Tim-Philipp Müller [Tue, 25 Jul 2006 16:47:04 +0000 (16:47 +0000)]
tag: id3v2: Extract frames for ID3v2 versions prior to ID3v2.3.0 properly as well, and add the version to...

Original commit message from CVS:
* gst-libs/gst/tag/id3v2.c:
(id3demux_add_id3v2_frame_blob_to_taglist):
Extract frames for ID3v2 versions prior to ID3v2.3.0 properly as
well, and add the version to the blob's buffer caps, since that
information will be needed for deserialisation later on (#348644).

13 years agotag: id3v2: On second thought, it might be wiser and more efficient not to do tag...
Tim-Philipp Müller [Sun, 23 Jul 2006 11:33:54 +0000 (11:33 +0000)]
tag: id3v2: On second thought, it might be wiser and more efficient not to do tag registration from a streaming th...

Original commit message from CVS:
* gst-libs/gst/tag/gstid3demux.c: (plugin_init):
* gst-libs/gst/tag/id3v2.c:
(id3demux_add_id3v2_frame_blob_to_taglist):
* gst-libs/gst/tag/id3v2.h:
On second thought, it might be wiser and more efficient
not to do tag registration from a streaming thread.

13 years agotag: id3v2: Put ID3v2 frames we can't parse as binary blobs into private tags, so...
Tim-Philipp Müller [Sun, 23 Jul 2006 10:56:27 +0000 (10:56 +0000)]
tag: id3v2: Put ID3v2 frames we can't parse as binary blobs into private tags, so that they are not lost ...

Original commit message from CVS:
* gst-libs/gst/tag/id3v2.c:
(id3demux_add_id3v2_frame_blob_to_taglist),
(id3demux_id3v2_frames_to_tag_list):
Put ID3v2 frames we can't parse as binary blobs into private
tags, so that they are not lost when retagging, at least once
id3v2mux has been taught to re-inject those frames again.
See bug #334375.

13 years agotag: id3v2: Don't use \n in debug lines
Wim Taymans [Fri, 21 Jul 2006 10:57:00 +0000 (10:57 +0000)]
tag: id3v2: Don't use \n in debug lines

Original commit message from CVS:
* gst/avi/gstavidemux.c: (gst_avi_demux_parse_stream),
(gst_avi_demux_process_next_entry):
Fix some leaks.
* gst-libs/gst/tag/id3v2.c: (id3demux_id3v2_frames_to_tag_list):
Don't use \n in debug lines.

13 years agotag: id3v2: Set image type from APIC frame as "image-type" field of GST_TAG_IMAGE...
Tim-Philipp Müller [Thu, 22 Jun 2006 12:17:13 +0000 (12:17 +0000)]
tag: id3v2: Set image type from APIC frame as "image-type" field of GST_TAG_IMAGE buffer caps (#344605).

Original commit message from CVS:
* gst-libs/gst/tag/id3v2frames.c: (parse_picture_frame):
Set image type from APIC frame as "image-type" field
of GST_TAG_IMAGE buffer caps (#344605).