tree 763e8f4c7323d9cc414f6dc64dee5bb14954816e
parent 08ee9e20e7ad96ad748bd359ac5892f356cd9631
author Erwin Pan <erwinpan1@gmail.com> 1689265189 +0800
committer GitHub <noreply@github.com> 1689265189 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJksCQlCRBK7hj4Ov3rIwAAPCwIADJUKDpnAIkR9/i94x9YpAYr
 zVdWfa+fs5jrLFzkz5w7qF6CZMtBxirXL6AiAZZ+Koo6Vt2nqMDgqt3fEweEZxFt
 fFdHoNNnEllghRV24c7L4cTIWM5eEMq7j59qJWdGvkyisjVE0B6M7Vtq1qrh+0wN
 d0BbnRDsceo8POtkx1o8Vq7Iol20Vz3nHq6uL5XhtOWWg+8bvvQogH1PPZxFZ/+s
 vChWLCHkolLcPSZWscWzNrXeQ3w8OHuCp8ZzoavhBG1CcyrHI7HsG4RGjiiVhCbl
 OUFGCK6RG+XcJC/nW3yPB1vnARbORWmyl4n1DhCgK9ZHBt15bTL0zcwTc8l2XeU=
 =KM0T
 -----END PGP SIGNATURE-----
 

Workaround Fan PercentSetting floating point precision error after "ceil()" (#27912)

* Workaround Fan Percent Setting floating point err

Workaround floating point precision error which will cause invalid value
after ceil()

For example, the current value:
   speedMax: 10
   percent: 70
   speedMax * (percent * 0.01) = 7.000000000000001 (floating point
precision error)
   ceil(speedMax * (percent * 0.01)) = 8 => The error propagate to ceil
and cause the final result error.

* Restyled by clang-format

* Workaround Fan PercentSetting float error (2nd)

Use integer multiply & devide to workaround floating
point precision error which causes incorrect Fan PercentSetting
value after ceil calculation.

* Update src/app/clusters/fan-control-server/fan-control-server.cpp

Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>

---------

Co-authored-by: Restyled.io <commits@restyled.io>
Co-authored-by: Andrei Litvin <andy314@gmail.com>
Co-authored-by: Boris Zbarsky <bzbarsky@apple.com>