Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1dd068cd authored by Eino-Ville Talvala's avatar Eino-Ville Talvala Committed by android-build-merger
Browse files

CameraBase: Don't return an sp<> by reference am: 4b49489c am: 8c32699c... 

CameraBase: Don't return an sp<> by reference am: 4b49489c am: 8c32699c am: 8806c2a1 am: 99c4dd2f
am: dd82ab23

Change-Id: I068a2fbc0e5bf20adb92ac4e95817899a3d53298
parents 08ccbcc1 dd82ab23

camera/CameraBase.cpp

+5 −5
Original line number Diff line number Diff line
@@ -66,7 +66,7 @@ namespace { 


// establish binder interface to camera service

template <typename TCam, typename TCamTraits>

const sp<ICameraService>& CameraBase<TCam, TCamTraits>::getCameraService()

const sp<ICameraService> CameraBase<TCam, TCamTraits>::getCameraService()

{

    Mutex::Autolock _l(gLock);

    if (gCameraService.get() == 0) {
@@ -99,7 +99,7 @@ sp<TCam> CameraBase<TCam, TCamTraits>::connect(int cameraId, 
    sp<TCam> c = new TCam(cameraId);

    sp<TCamCallbacks> cl = c;

    status_t status = NO_ERROR;

    const sp<ICameraService>& cs = getCameraService();

    const sp<ICameraService> cs = getCameraService();



    if (cs != 0) {

        TCamConnectService fnConnectService = TCamTraits::fnConnectService;
@@ -196,7 +196,7 @@ int CameraBase<TCam, TCamTraits>::getNumberOfCameras() { 
template <typename TCam, typename TCamTraits>

status_t CameraBase<TCam, TCamTraits>::getCameraInfo(int cameraId,

                               struct CameraInfo* cameraInfo) {

    const sp<ICameraService>& cs = getCameraService();

    const sp<ICameraService> cs = getCameraService();

    if (cs == 0) return UNKNOWN_ERROR;

    return cs->getCameraInfo(cameraId, cameraInfo);

}
@@ -204,7 +204,7 @@ status_t CameraBase<TCam, TCamTraits>::getCameraInfo(int cameraId, 
template <typename TCam, typename TCamTraits>

status_t CameraBase<TCam, TCamTraits>::addServiceListener(

                            const sp<ICameraServiceListener>& listener) {

    const sp<ICameraService>& cs = getCameraService();

    const sp<ICameraService> cs = getCameraService();

    if (cs == 0) return UNKNOWN_ERROR;

    return cs->addListener(listener);

}
@@ -212,7 +212,7 @@ status_t CameraBase<TCam, TCamTraits>::addServiceListener( 
template <typename TCam, typename TCamTraits>

status_t CameraBase<TCam, TCamTraits>::removeServiceListener(

                            const sp<ICameraServiceListener>& listener) {

    const sp<ICameraService>& cs = getCameraService();

    const sp<ICameraService> cs = getCameraService();

    if (cs == 0) return UNKNOWN_ERROR;

    return cs->removeListener(listener);

}

include/camera/CameraBase.h

+1 −1
Original line number Diff line number Diff line
@@ -101,7 +101,7 @@ protected: 
    virtual void                     binderDied(const wp<IBinder>& who);



    // helper function to obtain camera service handle

    static const sp<ICameraService>& getCameraService();

    static const sp<ICameraService> getCameraService();



    sp<TCamUser>                     mCamera;

    status_t                         mStatus;