path: root/src/gui/painting/qpagedpaintdevice.cpp
diff options
authorMarc Mutz <>2015-06-01 00:22:35 +0200
committerMarc Mutz <>2015-06-03 12:02:55 +0000
commit133e40283155852585b5fc409909a00e920cfcfc (patch)
tree741cb5b83260c70f7fc37ef1a83dac4d4f37c493 /src/gui/painting/qpagedpaintdevice.cpp
parentb295afb064d728d8f59d0129979c019761624bca (diff)
De-duplicate vtables, part I: exported private classes
By making the destructor (usually the first non-inline, non-pure, virtual function, and therefore the trigger for most compilers to emit the vtable and type_info structures for the class in that TU) out-of-line, vtables and, more importantly, type_info structures for the class are pinned to a single TU. This prevents false negative dynamic_cast and catch evaluation. In this first batch, we de-inline destructors of exported private classes. Since they are already exported, users of these classes are unaffected by the change, and since it's private API, we don't need to avoid adding code to the out-of-line destructor until Qt 6. Change-Id: I450707877d2cb6a77f79ae1dd355facb98d6c517 Reported-by: Volker Krause <> Task-number: QTBUG-45582 Reviewed-by: Oswald Buddenhagen <> Reviewed-by: Friedemann Kleint <> Reviewed-by: Lars Knoll <> Reviewed-by: Konstantin Ritt <> Reviewed-by: Thiago Macieira <>
Diffstat (limited to 'src/gui/painting/qpagedpaintdevice.cpp')
1 files changed, 4 insertions, 0 deletions
diff --git a/src/gui/painting/qpagedpaintdevice.cpp b/src/gui/painting/qpagedpaintdevice.cpp
index 3adc5e9411..547cedf51f 100644
--- a/src/gui/painting/qpagedpaintdevice.cpp
+++ b/src/gui/painting/qpagedpaintdevice.cpp
@@ -36,6 +36,10 @@
\class QPagedPaintDevice
\inmodule QtGui