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

Commit 0dcc9bb3 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 am: 1dd068cd am: edf9d2f4
am: 6ddd08e3

Change-Id: Id948f785f4f68a043747c9df5f15ebf253985ad1
parents 245b5723 6ddd08e3

camera/CameraBase.cpp

+5 −5
Original line number Diff line number Diff line
@@ -65,7 +65,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) {
@@ -98,7 +98,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;
@@ -195,7 +195,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);

}
@@ -203,7 +203,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);

}
@@ -211,7 +211,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;