[gstreamer-bugs] [Bug 613690] [xmp] refactoring to 1-n tag mappings

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=613690
  GStreamer | gst-plugins-base | git

Thiago Sousa Santos <thiago.sousa.santos> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |0.10.29

--- Comment #14 from Thiago Sousa Santos <thiago.sousa.santos at collabora.co.uk> 2010-03-24 18:55:46 UTC ---
Module: gst-plugins-base
Branch: master
Commit: e82414643caf876f094f78a8df2293068fe11f13
URL:   
http://cgit.freedesktop.org/gstreamer/gst-plugins-base/commit/?id=e82414643caf876f094f78a8df2293068fe11f13

Author: Thiago Santos <thiago.sousa.santos at collabora.co.uk>
Date:   Sat Mar 20 11:17:38 2010 -0300

tag: xmp: Refactor mappings storage

This commit is only refactoring, no fetaures added.

Do not store tags in flexible arrays as it doesn't allow us
to use nested flexible arrays. This is going to be needed in the
following commits to map gst tags that are stored into
2 separate tags in xmp (Not that they are alternatives, but
they are complementary).

For example, GST_TAG_ELEVATION is represented in the exif
schema with 2 fields: the absolute altitude and an integer
to indicate if it is above or below sea level.

The previous mappings storage wouldn't allow us to
express it.

Also store a serialization and a deserialization function
for each xmp tag as some of them require some non-trivial
convertion to its string form.

Fixes #613690


Module: gst-plugins-base
Branch: master
Commit: fe1f3e35957ddae12ae662e8e950bd8eb2358520
URL:   
http://cgit.freedesktop.org/gstreamer/gst-plugins-base/commit/?id=fe1f3e35957ddae12ae662e8e950bd8eb2358520

Author: Thiago Santos <thiago.sousa.santos at collabora.co.uk>
Date:   Mon Mar 22 22:03:09 2010 -0300

tag: xmp: Refactor buffer parsing

When parsing the xmp buffer into the gst taglist store the
found tags into a list to be parsed only after finding all
tags on the buffer. This allows the parser function to search
this list for complimentary tags that should be parsed together

Fixes #613690


Module: gst-plugins-base
Branch: master
Commit: 7ebbfbd3a5571c0e7518c5de9e710b9b9915f91e
URL:   
http://cgit.freedesktop.org/gstreamer/gst-plugins-base/commit/?id=7ebbfbd3a5571c0e7518c5de9e710b9b9915f91e

Author: Thiago Santos <thiago.sousa.santos at collabora.co.uk>
Date:   Mon Mar 22 15:18:28 2010 -0300

tag: xmp: Adds mappings for LATITUDE and LONGITUDE

Adds the mappings for those tags and tests
for tags serialization.

Fixes #613690


Module: gst-plugins-base
Branch: master
Commit: e207463582cc227b73922e8d6a70a721b75baf45
URL:   
http://cgit.freedesktop.org/gstreamer/gst-plugins-base/commit/?id=e207463582cc227b73922e8d6a70a721b75baf45

Author: Thiago Santos <thiago.sousa.santos at collabora.co.uk>
Date:   Tue Mar 23 09:48:19 2010 -0300

tag: xmp: Fix off by one

Avoid ignoring single char tags, like exif:GPSAltitudeRef

Fixes #613690


Module: gst-plugins-base
Branch: master
Commit: 007bf4fe7cef2c911a280a8f2c5caa27f9f25c14
URL:   
http://cgit.freedesktop.org/gstreamer/gst-plugins-base/commit/?id=007bf4fe7cef2c911a280a8f2c5caa27f9f25c14

Author: Thiago Santos <thiago.sousa.santos at collabora.co.uk>
Date:   Tue Mar 23 09:32:40 2010 -0300

tag: xmp: Add Elevation tag mapping

Adds a mapping to the _ELEVATION tag, this is a different
mapping as it has to be mapped into exif:GPSAltitude and
exif:GPSAltitudeRef at the same time. So we needed to refactor
a little more to be able to deserialize it properly.

Now, when parsing a xmp buffer into a taglist all tags are
added to a list before being parsed so that when one of the
altitude tags are found the deserialization function can search
for its complementary tag to do the correct parsing

Fixes #613690

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.