Skip Menu |
 

This queue is for tickets about the Image-ExifTool CPAN distribution.

Report information
The Basics
Id: 79395
Status: resolved
Worked: 5 min
Priority: 0/
Queue: Image-ExifTool

People
Owner: EXIFTOOL [...] cpan.org
Requestors: xstej70 [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 9.04



Subject: Bug: Samsung NX lens database is incomplete
Date: Tue, 04 Sep 2012 13:41:02 +0200
To: bug-Image-ExifTool [...] rt.cpan.org
From: Jaroslav Stepanek <xstej70 [...] gmail.com>
Download (untitled) / with headers
text/plain 1.1k
Hello, I just did some research on the Samsung NX lenses in the exiv2 source code. Two lenses from the current portfolio were missing so I added them and I've also rewritten the names of other lenses (from samsungmn.cpp): { 0, N_("Built-in") }, { 1, N_("Samsung NX 30mm F2 Pancake") }, { 2, N_("Samsung NX 18-55mm F3.5-5.6 OIS") }, { 3, N_("Samsung NX 50-200mm F4-5.6 ED OIS") }, { 4, N_("Samsung NX 20-50mm F3.5-5.6 ED") }, { 6, N_("Samsung NX 18-200mm F3.5-6.3 ED OIS") }, { 5, N_("Samsung NX 20mm F2.8 Pancake") }, { 7, N_("Samsung NX 60mm F2.8 Macro ED OIS SSA") }, { 8, N_("Samsung NX 16mm F2.4 Pancake") }, { 9, N_("Samsung NX 85mm F1.4 ED SSA") } The original exiv2 list is the same as in exiftool. I don't know whether the changes will get accepted in exiv2 (I don't see why not) but will this list make it to exiftool as well? I am also planning on updating the list as soon as new lenses are released (if I can come by a SRW file to check the ID). Regards, Jaroslav Stepanek
Download (untitled) / with headers
text/plain 109b
Excellent, thanks! I have added the 2 new entries. For now I will leave the "NX" out of the names. - Phil
The lenses were renamed as suggested in ExifTool 9.03.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.