cyotek / imagelistview

Automatically exported from code.google.com/p/imagelistview
5 stars 0 forks source link

Please revert Revision: r436 #102

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Hi Ozgur,

On most Camera/Photo websites the most important parameters are:
Camera model, Exposure time, FNumber, ISO speed and Focal length.

I intended to use the MetadataExtractor class directly. (Focal Length was there 
for a reason and it doesn't do any harm.) Of course, I could always copy the 
class and change it to my liking again. But it would be nice if general utility 
classes are as open for external use as possible (e.g. more public/protected 
properties instead of internal/private all the time).

Thanks,
Jens

Original issue reported on code.google.com by modernv...@arcor.de on 29 Sep 2010 at 5:42

GoogleCodeExporter commented 9 years ago
I changed the header to feature request before even entering the subject. Seems 
it didn't work.

Original comment by modernv...@arcor.de on 29 Sep 2010 at 5:44

GoogleCodeExporter commented 9 years ago
Hi Jens,

I rolled back the change and added the FocalLength property to items.

Thanks,
Ozgur

Original comment by oozcitak on 29 Sep 2010 at 8:04

GoogleCodeExporter commented 9 years ago
Fixed in r439.

Original comment by oozcitak on 29 Sep 2010 at 8:04