Uploaded image for project: 'Qt for MCUs'
  1. Qt for MCUs
  2. QTMCU-384

Qul::Property's in Qul::Object class cause clang-tidy issues

    XMLWordPrintable

Details

    • Bug
    • Resolution: Incomplete
    • Not Evaluated
    • None
    • Qt for MCUs 2.8
    • QUL Team
    • None

    Description

      If we have a class that inherits from Qul::Object, and if we create Qul:Property in public members of the class, and if we run clang-tidy we get the following clang-tidy warning: 

      /Projects/ucl_minigui/SecondLevelTileHandler.hpp:20:24: error: member variable 'currentIndex' has public visibility [cppcoreguidelines-non-private-member-variables-in-classes,-warnings-as-errors]
          Qul::Property<int> currentIndex;
       

      The fix for this issue would be to move the Qul:Property to private. Which dosen't make sense, since we expect Qt properties to be public. 

      Attachments

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

        Activity

          People

            Unassigned Unassigned
            tam5mu Maham Tanveer
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes