D21830: Format photo exposure bias value
Stefan BrĂ¼ns
noreply at phabricator.kde.org
Sat Jun 15 12:51:04 BST 2019
bruns added a comment.
In D21830#480185 <https://phabricator.kde.org/D21830#480185>, @astippich wrote:
> In D21830#480181 <https://phabricator.kde.org/D21830#480181>, @bruns wrote:
>
> > Many cameras can be switched to 1/2 EV steps.
>
>
> Damn ;) do you know any other values? In case not, I would add this case in a loop and stupidly try everything
I think with 1/2 and 1/3 you have covered 99.9% of the cameras.
You can quantize to 1/6, and if the fractional part is in {2,3,4}, use the "%1 %2/%3" format.
To avoid the `if() { if() { if() { }}}` cascade, you may use a nested function (lambda) for the default case (format as double), and do an early return.
How about integral/fractional instead of wholeNumber/remainder?
REPOSITORY
R286 KFileMetaData
REVISION DETAIL
https://phabricator.kde.org/D21830
To: astippich, bruns, ngraham
Cc: kde-frameworks-devel, #baloo, LeGast00n, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190615/d4dca60e/attachment.html>
More information about the Kde-frameworks-devel
mailing list