diff options
| author | XpLoDWilD <xplodgui@gmail.com> | 2012-12-11 22:08:31 +0000 |
|---|---|---|
| committer | XpLoDWilD <xplodgui@gmail.com> | 2012-12-11 22:22:26 +0000 |
| commit | 95577f87b005857b47f4e74e4d9a4a1d3ed778ea (patch) | |
| tree | a6cc854d7fbf41b39e79ba984a6018dda051045b /camerawrapper | |
| parent | 4f0295469dd6064effb701ea7c47432297589f3f (diff) | |
| download | device_samsung_smdk4412-common-95577f87b005857b47f4e74e4d9a4a1d3ed778ea.tar.gz device_samsung_smdk4412-common-95577f87b005857b47f4e74e4d9a4a1d3ed778ea.tar.bz2 device_samsung_smdk4412-common-95577f87b005857b47f4e74e4d9a4a1d3ed778ea.zip | |
smdk4412-common: camera: Enable preview handling, and don't report take_picture errors
See CameraWrapper::camera_take_picture comments
Change-Id: I3255424dd946b1134fdf0cf6539cc73bc9b40bcf
Diffstat (limited to 'camerawrapper')
| -rw-r--r-- | camerawrapper/CameraWrapper.cpp | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/camerawrapper/CameraWrapper.cpp b/camerawrapper/CameraWrapper.cpp index 18f6da2..b6a5c88 100644 --- a/camerawrapper/CameraWrapper.cpp +++ b/camerawrapper/CameraWrapper.cpp @@ -322,7 +322,13 @@ int camera_take_picture(struct camera_device * device) if(!device) return -EINVAL; - return VENDOR_CALL(device, take_picture); + // We safely avoid returning the exact result of VENDOR_CALL here. If ZSL + // really bumps fast, take_picture will be called while a picture is already being + // taken, leading to "picture already running" error, crashing Gallery app. Afaik, + // there is no issue doing 0 (error appears in logcat anyway if needed). + VENDOR_CALL(device, take_picture); + + return 0; } int camera_cancel_picture(struct camera_device * device) |