Uploaded image for project: 'Qt Mobility'
  1. Qt Mobility
  2. QTMOBILITY-620 Location API
  3. QTMOBILITY-654

The QGeoSatelliteInfoSource signals should be merged

    XMLWordPrintable

Details

    • Change Request
    • Resolution: Unresolved
    • P3: Somewhat important
    • None
    • Location
    • None

    Description

      At present QGeoSatelliteInfoSource emits two signals - sattelitesInUseUpdated(QList<QGeoSatelliteInfo>) and satellitesInViewUpdated(QList<QGeoSatelliteInfo>).
      This is fine for when clients want to keep track of one or the other (or, more likely, one and then the other).

      If a client wants to keep track of both at the same time they will need to implement some complicated logic to keep the data coming from the two signals in sync. All implementations currently emit both signals at approximately the same time, although since we don't make any guarantees about that the clients need to handle a much more complicated case than they should.

      An alternative has been proposed, where QGeoSatelliteInfo would have an bool inUse() and void setInUse(bool) methods and the signals would be merge to satellitesUpdated(QList<QGeoSatelliteInfo>).

      This appears to be a superior solution to what we currently have, and would have made it into the 1.0 release if the suggestion had been brought up with time for us to make the change and test it properly.

      Attachments

        Activity

          People

            alewilso Alex Wilson (closed Nokia identity) (Inactive)
            minshin Minjung Shin (closed Nokia identity) (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:

              Time Tracking

                Estimated:
                Original Estimate - 2 weeks
                2w
                Remaining:
                Remaining Estimate - 2 weeks
                2w
                Logged:
                Time Spent - Not Specified
                Not Specified