diff options
author | Ahmad Samir <a.samirh78@gmail.com> | 2024-04-17 02:08:48 +0200 |
---|---|---|
committer | Ahmad Samir <a.samirh78@gmail.com> | 2024-04-20 04:29:31 +0200 |
commit | df6b74f4617bf0ba14420ed15fcc5866cccfb7d0 (patch) | |
tree | 42eb97c82d52e266888fcb7b5e2dba623215c662 /tests/auto/gui/kernel/qguivariant/no_application/main.cpp | |
parent | d691872b865c735f9d35886388c89f7ace7df5c8 (diff) |
QFSFileEngine: update d->fileEntry after a successful rename
Otherwise member methods such as size() could return incorrect info.
Add setFileEntry(QFileSystemEntry &&); to reuse a QFileSystemEntry if we
have already constructed one.
As a result of this, a QTemporaryFileEngine::setFileName() call has
become redundant and can be removed; the code it executed is already
taken care of:
- QFSFileEngine::close(): already called by QTFEngine::rename() a couple
of lines above
- QFSFileEngine::setFileName(): QTFEngine::rename() calls
QFSFileEngine::rename() which in turn updates the `fileEntry`
This commit is covered by tst_QTemporaryFile::rename(), i.e. if
QFSFileEngine::rename() didn't update the fileEntry, that test would
fail.
Change-Id: I312f35cf7fdf9b1a8cd0bce5e98ba7a48cf9426e
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@gmx.de>
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Diffstat (limited to 'tests/auto/gui/kernel/qguivariant/no_application/main.cpp')
0 files changed, 0 insertions, 0 deletions