diff options
| author | Ivan Solovev <ivan.solovev@qt.io> | 2025-07-15 16:27:39 +0200 |
|---|---|---|
| committer | Ivan Solovev <ivan.solovev@qt.io> | 2025-07-17 18:07:57 +0200 |
| commit | 70fa17fcb832f187bdaf0edad9cdcef32053e15f (patch) | |
| tree | 1883ad56234ea82025eebac21cc68fdb9fa2a452 /src/corelib/doc/snippets | |
| parent | aa8d2c237a6930fca8a5741a5589a0d58e198d20 (diff) | |
QDBusPendingReply: apply Rule Of Zero
The only non-default SMF was the copy-assignment operator, which was
calling the assign() function.
However, that call seems to be redundant, as the other object should
have already called it during construction, and since the implementation
uses QESDP, the copy-assignment should just increase the ref count.
The tests that were added in the previous patch confirm that the call
is not needed. This means that we can apply Rule Of Zero and remove
all SMFs.
Change-Id: I436dad79730b7a8d4ef50abedc09783588a803f0
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Reviewed-by: Marc Mutz <marc.mutz@qt.io>
Diffstat (limited to 'src/corelib/doc/snippets')
0 files changed, 0 insertions, 0 deletions
