Merge lp:~schwann/qtubuntu-camera/camera-torch-support into lp:qtubuntu-camera

Proposed by Günter Schwann
Status: Merged
Approved by: Günter Schwann
Approved revision: 54
Merged at revision: 62
Proposed branch: lp:~schwann/qtubuntu-camera/camera-torch-support
Merge into: lp:qtubuntu-camera
Diff against target: 30 lines (+4/-2)
1 file modified
src/aalcameraflashcontrol.cpp (+4/-2)
To merge this branch: bzr merge lp:~schwann/qtubuntu-camera/camera-torch-support
Reviewer Review Type Date Requested Status
Victor Tuson Palau (community) Approve
PS Jenkins bot continuous-integration Approve
Ubuntu Phablet Team Pending
Review via email: mp+168468@code.launchpad.net

Commit message

Support FlashTorch

Description of the change

Support FlashTorch
This enable support for the torch component as well.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Victor Tuson Palau (vtuson) wrote :

I try the deb but broke the cam and the app running via QTcreator gives the following error
/usr/lib/arm-linux-gnueabihf/qt5/bin/qmlscene: symbol lookup error: /usr/lib/arm-linux-gnueabihf/qt5/plugins/mediaservice/libaalcamera.so: undefined symbol: _ZN12SharedSignal8instanceEv

Revision history for this message
Günter Schwann (schwann) wrote :

Ok - so we need to wait for an image including the new libhybris (with video support)

Revision history for this message
Victor Tuson Palau (vtuson) wrote :

> Ok - so we need to wait for an image including the new libhybris (with video
> support)
please ping me in irc when this us ready to test. thanks

53. By Günter Schwann

Trunk merged

Revision history for this message
Günter Schwann (schwann) wrote :

Victor - it's fixed now. You can test it again (didn't see you on IRC ;)

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Günter Schwann (schwann) wrote :

The debian package of the last build here on the saucy image-13 works fine for me.

54. By Günter Schwann

Trunk merged

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Victor Tuson Palau (vtuson) wrote :

using the latest build Camera.FlashTorch works fine.

Revision history for this message
Victor Tuson Palau (vtuson) wrote :

WFM!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/aalcameraflashcontrol.cpp'
--- src/aalcameraflashcontrol.cpp 2013-06-05 16:11:12 +0000
+++ src/aalcameraflashcontrol.cpp 2013-07-02 13:37:08 +0000
@@ -39,7 +39,8 @@
39{39{
40 if (m_service->isBackCameraUsed()) {40 if (m_service->isBackCameraUsed()) {
41 if (mode==QCameraExposure::FlashAuto || mode==QCameraExposure::FlashOff ||41 if (mode==QCameraExposure::FlashAuto || mode==QCameraExposure::FlashOff ||
42 mode==QCameraExposure::FlashOn || mode==QCameraExposure::FlashVideoLight) {42 mode==QCameraExposure::FlashOn || mode==QCameraExposure::FlashVideoLight ||
43 mode==QCameraExposure::FlashTorch) {
43 return true;44 return true;
44 }45 }
45 } else {46 } else {
@@ -98,6 +99,7 @@
98 case QCameraExposure::FlashOn:99 case QCameraExposure::FlashOn:
99 return FLASH_MODE_ON;100 return FLASH_MODE_ON;
100 case QCameraExposure::FlashVideoLight:101 case QCameraExposure::FlashVideoLight:
102 case QCameraExposure::FlashTorch:
101 return FLASH_MODE_TORCH;103 return FLASH_MODE_TORCH;
102 case QCameraExposure::FlashAuto:104 case QCameraExposure::FlashAuto:
103 default:105 default:
@@ -113,7 +115,7 @@
113 case FLASH_MODE_ON:115 case FLASH_MODE_ON:
114 return QCameraExposure::FlashOn;116 return QCameraExposure::FlashOn;
115 case FLASH_MODE_TORCH:117 case FLASH_MODE_TORCH:
116 return QCameraExposure::FlashVideoLight;118 return QCameraExposure::FlashTorch;
117 case FLASH_MODE_AUTO:119 case FLASH_MODE_AUTO:
118 default:120 default:
119 return QCameraExposure::FlashAuto;121 return QCameraExposure::FlashAuto;

Subscribers

People subscribed via source and target branches