bruns added inline comments.

INLINE COMMENTS

> exiv2extractor.cpp:299
>      Exiv2::ExifData::const_iterator it = 
> data.findKey(Exiv2::ExifKey("Exif.GPSInfo.GPSAltitude"));
> -    if (it != data.end() && (it->value().typeId() == Exiv2::unsignedRational 
> || it->value().typeId() == Exiv2::signedRational)) {
> +    if (it != data.end() && it->count() > 0 &&
> +        (it->value().typeId() == Exiv2::unsignedRational || 
> it->value().typeId() == Exiv2::signedRational)) {

`!it->empty()` ?

REPOSITORY
  R286 KFileMetaData

REVISION DETAIL
  https://phabricator.kde.org/D20877

To: astippich, bruns
Cc: kde-frameworks-devel, #baloo, gennad, domson, ashaposhnikov, michaelh, 
astippich, spoorun, ngraham, bruns, abrahams

Reply via email to