[camera] Camera Example app's setExposureOffset is not robust #154132
Labels
d: examples
Sample code and demos
p: camera
The camera plugin
P3
Issues that are less important to the Flutter project
package
flutter/packages repository. See also p: labels.
team-ecosystem
Owned by Ecosystem team
triaged-ecosystem
Triaged by Ecosystem team
What package does this bug report belong to?
camera
What target platforms are you seeing this bug on?
Android
Have you already upgraded your packages?
Yes
Dependency versions
pubspec.lock
Steps to reproduce
setExposureOffset
_currentExposureOffset = offset;
within asetState
however the effective setting could fail at which point the_currentExposureOffset
will not reflect the actual state.setFlashMode
orsetExposureMode
(and alike) - which don't have a return value - thesetExposureOffset
has a return value. The return value would be the effective offset and the result of the operation and that should be the new value of_currentExposureOffset
. Assigning it to the input argument doesn't make sense and fails the VGV code analysis.setExposureOffset
doesn't throw an exception it may round the offset, so we better take that as the new value.Expected results
Handle if the
setExposureOffset
would throw a CameraException exception and also treat if thesetExposureOffset
would round the offset, try to store the proper new offset value.Actual results
Right now if any rounding or exception would happen the internal state may diverge compared to the actual offset.
Code sample
Code sample
Flutter Doctor output
Doctor output
The text was updated successfully, but these errors were encountered: