[Qt-interest] New image metadata parser based on the Qt

[Sami Maisniemi]

Thank you for your feedback. I have pushed the changes to the version control.

-Sami

On Mar 31, 2011, at 10:10 AM, Miroslav Rajcic wrote:

> Classes seem to work fine after several fixes:
> 
> - compiler (Visual Studio 2008) complains (warning) that the following operator
> is recursive and it will generate stack overflow:
> 
>  QImageMetaData::operator=
> 
> If you remove the first line in the operator code, it will work fine.
> 
> - the same compiler generates an error for the following method:
> 
>  QImageMetaData::readIFD
> 
> You are using large number of nested "if/else" loops and compiler generates this
> error: C1061: compiler limit : blocks nested too deeply
> 
> If you convert the "if" loops starting on the line 2765 in the "readIFD"
> function to "switch/case" statements, the problem will be solved
> 
> - third problem I noticed is that reading exif info blocks forever on the jpg
> images that do not contain exif data at all
> This is solved by patching the method "QImageMetaData::identify" by adding the
> check "if(length < 1)  break;" after the line "*input >> length;"
> 
> I will send you my patched version by mail.
> Thank you for your work
> 
> _______________________________________________
> Qt-interest mailing list
> Qt-interest at qt.nokia.com
> http://lists.qt.nokia.com/mailman/listinfo/qt-interest

Sami Maisniemi | Imaging Specialist | Kate Labs Ltd
www.katelabs.com | +358 400 114 070