Things To Look Out For In Reviews

From Qt Wiki
Revision as of 09:27, 5 January 2024 by EdwardWelbourne (talk | contribs) (bah - too many colons.)
Jump to navigation Jump to search


This page is a low-entry-bar staging area for potential project-wide guidelines. After each release, we should go over each of these and decide whether we keep them here, or, preferably, fold them into Review Policy, API Design Principles, Qt Coding Style, Coding Conventions or a new or existing QUIP.

Newly-Added Things

When adding a new Thing, don't forget to provide rationale.

Enums

New Classes

Make sure new classes follow the draft value-class mechanics QUIP-22.

Polymorphic Classes

- Make dtor out-of-line, even if empty (then =default it). Rationale: pins the vtable to a single TU (and prevents -Wweak-vtable warnings). References: https://bugreports.qt.io/browse/QTBUG-45582

Decided To Keep Here

(make a proper subsection grouping when moving stuff here)