diff options
| author | Marc Mutz <marc.mutz@qt.io> | 2023-02-24 08:50:14 +0100 |
|---|---|---|
| committer | Marc Mutz <marc.mutz@qt.io> | 2023-03-17 00:41:32 +0100 |
| commit | ca1b03be697176233a0680aeab37f4161599b6cc (patch) | |
| tree | 048537ad57eb9970b2237ccc4faeac1e4d0914ff /tests/auto/sql/kernel/qsqldatabase/tst_qsqldatabase.cpp | |
| parent | 4ac7ac5b595cb1710a63d4dc917e65e20baca7a1 (diff) | |
QCryptographicHash: split initialization from reset()ing
Before the OpenSSL3 mode was added, these operations were the
same. The placement-new of the active union member was arguably not
even necessary, as just naming it should start their lifetime;
certainly all initialization was vacuous, anyway. So it didn't really
matter that we re-did it on each reset().
But OpenSSL3 mode is different: initialization allocates (and can
fail) whereas reset() does not¹, so it makes sense to separate them,
so as to re-establish the lost proper noexcept'ness of reset().
¹ or so we assume
This will also help when moving the EVP members into the union later.
Pick-to: 6.5
Change-Id: Id1c9a00cddb95fb2235162409c2301dd4c35228a
Reviewed-by: Mårten Nordheim <marten.nordheim@qt.io>
Diffstat (limited to 'tests/auto/sql/kernel/qsqldatabase/tst_qsqldatabase.cpp')
0 files changed, 0 insertions, 0 deletions
