bruns added inline comments.
INLINE COMMENTS
> propertyinfotest.cpp:144
> { Property::AspectRatio, true, 1.77778,
> QStringLiteral("1.78:1")},
> + { Property::PhotoFocalLength, true, 2.0,
> QStringLiteral("2 mm")},
> };
For small frame cameras (compact cameras, smartphones, ...) the real focal
length is often in the range of only a few millimeters, please add a test case
which checks e.g. "2.4 mm" is formatted as such.
REPOSITORY
R286 KFileMetaData
BRANCH
focal_length
REVISION DETAIL
https://phabricator.kde.org/D21613
To: astippich, bruns, ngraham
Cc: kde-frameworks-devel, #baloo, LeGast00n, domson, ashaposhnikov, michaelh,
astippich, spoorun, ngraham, bruns, abrahams
