Uploaded image for project: 'Qt for Python'
  1. Qt for Python
  2. PYSIDE-743

shiboken: Check type system attributes

    Details

    • Type: Task
    • Status: Closed
    • Priority: P2: Important
    • Resolution: Done
    • Affects Version/s: 5.11.0
    • Fix Version/s: 5.12.0
    • Component/s: Shiboken
    • Labels:
      None
    • Technical Risk:
      Normal

      Description

      shiboken parses a number of typesystem attributes which appear not to be used later on (according to code model)

      Modifications

      Name Code Value used in shiboken Used in Qt typesys XML Documented Comment
      modify-argument/reference/add-all ReferenceCount/Action no no yes Would make sense?
      modify-argument/reference/ignore   no no yes Intended as no-op?
      modify-argument/no-null-pointer/default-value ArgumentModification.nullPointerDefaultValue no no no  
      modify-function/access/final, non-final AbstractMetaAttributes no yes no Unclear purpose
      modify-function/access/friendly AbstractMetaAttributes no no yes Unclear purpose
      modify-function/allow-thread FunctionModification::allowThread() no yes no Unclear purpose
      modify-function/thread FunctionModification::isThread() no no no Unclear purpose
      modify-function/associated-to FunctionModification::association no no no Unclear purpose
      modify-function/virtual-slot AbstractMetaFunction::isVirtualSlot questionable no no Unclear purpose
      custom-de/constructor TypeEntry::customDeConstructor most likely not no no  
      argument-map CodeSnip::ArgumentMap most likely not no no  
      modify-field/read/write FieldModification::isReadable, Readable no yes yes readOnly would make sense?
      enum-type/reject-enum-value EnumTypeEntry/addEnumValueRejection no no yes bug

      Types

      Name Code Value used in shiboken Used in Qt typesys XML Documented Comment
      primitive-type/preferred-conversion TypeEntry::preferredConversion() no no yes Contradictory docs (bool vs text)
      *revision, enum-type/flags-revision getTypeRevision() no yes yes apparently same as since, just using one integer
      enum-type/upper,lower-bound EnumTypeEntry() getter no yes yes  
      enum-type/ force-integer EnumTypeEntry() getter no yes yes  
      enum-type/ extensible EnumTypeEntry() getter no yes yes  
      object-type/force-abstract ComplexTypeEntry::TypeFlag no yes yes  
      object-type/generic ComplexTypeEntry::isGenericClass() questionable no no  
      object-type/held-type ComplexTypeEntry::heldTypeValue() no no no  
      *-type/delete-in-main-thread ComplexTypeEntry::typeFlags no no no  

        Attachments

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

          Activity

            People

            • Assignee:
              kleint Friedemann Kleint
              Reporter:
              kleint Friedemann Kleint
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: