After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 767881 - tag: add GST_TAG_CAPTURING_FOCAL_LENGTH_35mm
tag: add GST_TAG_CAPTURING_FOCAL_LENGTH_35mm
Status: RESOLVED DUPLICATE of bug 753930
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
1.x
Other Linux
: Normal enhancement
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2016-06-20 17:17 UTC by Paulo Neves
Modified: 2016-06-21 14:37 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch for the bug description (3.65 KB, patch)
2016-06-20 17:17 UTC, Paulo Neves 		none Details | Review
Added GST_TAG_CAPTURING_FOCAL_LENGTH_35MM (3.65 KB, patch)
2016-06-20 18:43 UTC, Paulo Neves 		none Details | Review

Description Paulo Neves 2016-06-20 17:17:26 UTC 
Created attachment 330097 [details] [review]
Patch for the bug description

Very simple patch.

Added the 35mm film equivalent length
GST_TAG_CAPTURING_FOCAL_LENGTH_35mm that
exists in the exif standard. This patch
enables the parsing of the exif equivalent
tag.

This tag is important because a number of
photogrametry applications use it to
calculate the sensor size of the camera,
making pictures without this tag useless
for the purpose.

See https://support.pix4d.com/hc/en-us/articles/205732299-Standard-EXIF-tags-read-by-Pix4Dmapper#gsc.tab=0
Comment 1 Tim-Philipp Müller 2016-06-20 18:14:31 UTC 
Comment on attachment 330097 [details] [review]
Patch for the bug description

> GST_TAG_CAPTURING_FOCAL_LENGTH_35mm

Should probably be GST_TAG_CAPTURING_FOCAL_LENGTH_35MM ?
Comment 2 Paulo Neves 2016-06-20 18:43:53 UTC 
Created attachment 330100 [details] [review]
Added GST_TAG_CAPTURING_FOCAL_LENGTH_35MM

Argh sorry for the typo. MM are always lowercase and I got carried away. This patch has the macro names with 35MM upper cased. The only difference.
Comment 4 Tim-Philipp Müller 2016-06-20 22:28:16 UTC 
Looks like it doesn't it?

*** This bug has been marked as a duplicate of bug 753930 ***
Comment 5 Paulo Neves 2016-06-21 14:37:06 UTC 
Ouch. Sorry for the waste of time.