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

The QGeoSatelliteInfoSource signals should be merged

XMLWordPrintable

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

      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.

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            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

              Created:
              Updated:

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

                  There are no open Gerrit changes