Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-25885

Change the reading class to a value-based contained

    XMLWordPrintable

    Details

    • Type: User Story
    • Status: Withdrawn
    • Priority: P3: Somewhat important
    • Resolution: Out of scope
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Sensors
    • Labels:
      None

      Description

      Overview

      The C++ API will use a read-only value-based class with implicit sharing. No Q_OBJECT, no properties, no single-pointer-with-changing-data.

      The app is free to keep these classes around as long as it likes.

      Implementation Plan

      Create a class that uses QSharedDataPointer or similar. This class will be instantiated in the middleware and pushed into the FIFO before being taken out by the app.

      Notes

      The QML API requires a QObject-based class so wrapper(s) will need to be created in the QML plugin.

      Research Questions

      What does example app code look like?

      How can we unit test this?

      How can we document this?

      What does this hold? How does the backend/API handle this class?

      I don't think this can hold a SensorData struct because that kills the ability to add a new sensor type without recompiling existing code. Maybe a set of concrete classes or something with templates can help out here?

        Attachments

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

          Activity

            People

            Assignee:
            lpotter Lorn Potter
            Reporter:
            lramsay Lincoln Ramsay (closed Nokia identity) (Inactive)
            PM Owner:
            Veli-Pekka Heinonen Veli-Pekka Heinonen
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Gerrit Reviews

                There are no open Gerrit changes