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

Commit 4635555a authored by Steven Moreland's avatar Steven Moreland Committed by Gerrit Code Review
Browse files

Merge "libbinder_ndk: simplify AStatus get"

parents fe81bae6 85c138ae

libs/binder/ndk/parcel.cpp

+1 −1
Original line number Original line Diff line number Diff line
@@ -272,7 +272,7 @@ binder_status_t AParcel_readParcelFileDescriptor(const AParcel* parcel, int* fd) 
}

}





binder_status_t AParcel_writeStatusHeader(AParcel* parcel, const AStatus* status) {

binder_status_t AParcel_writeStatusHeader(AParcel* parcel, const AStatus* status) {

    return PruneStatusT(status->get()->writeToParcel(parcel->get()));

    return PruneStatusT(status->get().writeToParcel(parcel->get()));

}

}

binder_status_t AParcel_readStatusHeader(const AParcel* parcel, AStatus** status) {

binder_status_t AParcel_readStatusHeader(const AParcel* parcel, AStatus** status) {

    ::android::binder::Status bstatus;

    ::android::binder::Status bstatus;

libs/binder/ndk/status.cpp

+6 −6
Original line number Original line Diff line number Diff line
@@ -47,27 +47,27 @@ AStatus* AStatus_fromStatus(binder_status_t status) { 
}

}





bool AStatus_isOk(const AStatus* status) {

bool AStatus_isOk(const AStatus* status) {

    return status->get()->isOk();

    return status->get().isOk();

}

}





binder_exception_t AStatus_getExceptionCode(const AStatus* status) {

binder_exception_t AStatus_getExceptionCode(const AStatus* status) {

    return PruneException(status->get()->exceptionCode());

    return PruneException(status->get().exceptionCode());

}

}





int32_t AStatus_getServiceSpecificError(const AStatus* status) {

int32_t AStatus_getServiceSpecificError(const AStatus* status) {

    return status->get()->serviceSpecificErrorCode();

    return status->get().serviceSpecificErrorCode();

}

}





binder_status_t AStatus_getStatus(const AStatus* status) {

binder_status_t AStatus_getStatus(const AStatus* status) {

    return PruneStatusT(status->get()->transactionError());

    return PruneStatusT(status->get().transactionError());

}

}





const char* AStatus_getMessage(const AStatus* status) {

const char* AStatus_getMessage(const AStatus* status) {

    return status->get()->exceptionMessage().c_str();

    return status->get().exceptionMessage().c_str();

}

}





const char* AStatus_getDescription(const AStatus* status) {

const char* AStatus_getDescription(const AStatus* status) {

    android::String8 description = status->get()->toString8();

    android::String8 description = status->get().toString8();

    char* cStr = new char[description.size() + 1];

    char* cStr = new char[description.size() + 1];

    memcpy(cStr, description.c_str(), description.size() + 1);

    memcpy(cStr, description.c_str(), description.size() + 1);

    return cStr;

    return cStr;

libs/binder/ndk/status_internal.h

+1 −2
Original line number Original line Diff line number Diff line
@@ -25,8 +25,7 @@ struct AStatus { 
    AStatus() {}  // ok

    AStatus() {}  // ok

    explicit AStatus(::android::binder::Status&& status) : mStatus(std::move(status)) {}

    explicit AStatus(::android::binder::Status&& status) : mStatus(std::move(status)) {}





    ::android::binder::Status* get() { return &mStatus; }

    const ::android::binder::Status& get() const { return mStatus; }

    const ::android::binder::Status* get() const { return &mStatus; }





   private:

   private:

    ::android::binder::Status mStatus;

    ::android::binder::Status mStatus;