php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #23497 computation of 35mmFocalLength (exif)
Submitted: 2003-05-05 16:59 UTC Modified: 2005-02-01 22:54 UTC
Votes:1
Avg. Score:3.0 ± 0.0
Reproduced:0 of 0 (0.0%)
From: francois at guillier dot org Assigned:
Status: Wont fix Package: EXIF related
PHP Version: 4CVS-2003-05-05 (stable) OS:
Private report: No CVE-ID: None
View Add Comment Developer Edit
Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know!
Just going to say 'Me too!'? Don't clutter the database with that please — but make sure to vote on the bug!
Your email address:
MUST BE VALID
Solve the problem:
48 - 16 = ?
Subscribe to this entry?

 
 [2003-05-05 16:59 UTC] francois at guillier dot org 
In the following code (around ligne 3830 of exif.c):

exif_iif_add_fmt(&ImageInfo, SECTION_COMPUTED, "35mmFocalLength" TSRMLS_CC, "%dmm", (int)(ImageInfo.FocalLength/ImageInfo.CCDWidth*35+0.5));

Why is CCDWidth multiplied by 35 knowing than, on 35mm film,
the image actual size is 36x24mm?

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2003-05-05 18:16 UTC] helly@php.net 
This is an approximation that maybe is wrong but every manufacturer does its own error. So maybe the correct thing would be to provide an ini setting for that.

If you come up with a hint that shows that 36 is really better i consider changing it. Otherwise there will be now change before next major version.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Thu Apr 25 14:01:31 2024 UTC