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

Commit d2e8d0b5 authored by Treehugger Robot's avatar Treehugger Robot Committed by Gerrit Code Review
Browse files

Merge "[hardware][interfaces][gnss] fix -Wreorder-init-list"

parents b6fd785e 477c7e87

gnss/1.0/default/Gnss.cpp

+14 −14
Original line number Original line Diff line number Diff line
@@ -129,19 +129,19 @@ void Gnss::gnssSvStatusCb(GnssSvStatus* status) { 
        auto svInfo = status->gnss_sv_list[i];

        auto svInfo = status->gnss_sv_list[i];

        IGnssCallback::GnssSvInfo gnssSvInfo = {

        IGnssCallback::GnssSvInfo gnssSvInfo = {

                .svid = svInfo.svid,

                .svid = svInfo.svid,

            .constellation = static_cast<

                .constellation = static_cast<android::hardware::gnss::V1_0::GnssConstellationType>(

                android::hardware::gnss::V1_0::GnssConstellationType>(

                        svInfo.constellation),

                        svInfo.constellation),

                .cN0Dbhz = svInfo.c_n0_dbhz,

                .cN0Dbhz = svInfo.c_n0_dbhz,

                .elevationDegrees = svInfo.elevation,

                .elevationDegrees = svInfo.elevation,

                .azimuthDegrees = svInfo.azimuth,

                .azimuthDegrees = svInfo.azimuth,

                .carrierFrequencyHz = 0,

                // Older chipsets do not provide carrier frequency, hence

                // Older chipsets do not provide carrier frequency, hence

                // HAS_CARRIER_FREQUENCY flag and the carrierFrequencyHz fields

                // HAS_CARRIER_FREQUENCY flag and the carrierFrequencyHz fields

                // are not set. So we are resetting both fields here.

                // are not set. So we are resetting both fields here.

                .svFlag = static_cast<uint8_t>(

                .svFlag = static_cast<uint8_t>(

                svInfo.flags &= ~(static_cast<uint8_t>(

                        svInfo.flags &=

                    IGnssCallback::GnssSvFlags::HAS_CARRIER_FREQUENCY))),

                        ~(static_cast<uint8_t>(IGnssCallback::GnssSvFlags::HAS_CARRIER_FREQUENCY))),

            .carrierFrequencyHz = 0};

        };

        svStatus.gnssSvList[i] = gnssSvInfo;

        svStatus.gnssSvList[i] = gnssSvInfo;

    }

    }

gnss/2.0/default/GnssMeasurement.cpp

+3 −2
Original line number Original line Diff line number Diff line
@@ -119,12 +119,13 @@ GnssData GnssMeasurement::getMockMeasurement() { 
    V2_0::IGnssMeasurementCallback::GnssMeasurement measurement_2_0 = {

    V2_0::IGnssMeasurementCallback::GnssMeasurement measurement_2_0 = {

            .v1_1 = measurement_1_1,

            .v1_1 = measurement_1_1,

            .codeType = "C",

            .codeType = "C",

            .constellation = GnssConstellationType::GLONASS,

            .state = GnssMeasurementState::STATE_CODE_LOCK | GnssMeasurementState::STATE_BIT_SYNC |

            .state = GnssMeasurementState::STATE_CODE_LOCK | GnssMeasurementState::STATE_BIT_SYNC |

                     GnssMeasurementState::STATE_SUBFRAME_SYNC |

                     GnssMeasurementState::STATE_SUBFRAME_SYNC |

                     GnssMeasurementState::STATE_TOW_DECODED |

                     GnssMeasurementState::STATE_TOW_DECODED |

                     GnssMeasurementState::STATE_GLO_STRING_SYNC |

                     GnssMeasurementState::STATE_GLO_STRING_SYNC |

                     GnssMeasurementState::STATE_GLO_TOD_DECODED};

                     GnssMeasurementState::STATE_GLO_TOD_DECODED,

            .constellation = GnssConstellationType::GLONASS,

    };





    hidl_vec<IGnssMeasurementCallback::GnssMeasurement> measurements(1);

    hidl_vec<IGnssMeasurementCallback::GnssMeasurement> measurements(1);

    measurements[0] = measurement_2_0;

    measurements[0] = measurement_2_0;