diff options
| author | Ulf Hermann <ulf.hermann@qt.io> | 2022-03-28 08:57:05 +0200 |
|---|---|---|
| committer | Ulf Hermann <ulf.hermann@qt.io> | 2022-03-28 10:28:35 +0200 |
| commit | 664aa0c1893e03d4de44959f188f06082bcf5c0c (patch) | |
| tree | 96f681f78987682ffa69eca8d054f4f129d675dc /src/qml/jsruntime/qv4qobjectwrapper.cpp | |
| parent | e718e369a677fe57bcb0955bd26a20a4ef07069b (diff) | |
QQmlImport: Clarify addImportToNamespace
The code checker believes the import could leak there. Indeed there was
a hypothetical code path through the function that would leak: If all
existing imports were implicit and the new one was of low precedence.
Rephrase the code to account for this case and make it easier to see
that no code path returns the import without inserting into the list.
Change-Id: I4cd98633fd28d8f5fbc881851eda92f02c9cd1f9
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Diffstat (limited to 'src/qml/jsruntime/qv4qobjectwrapper.cpp')
0 files changed, 0 insertions, 0 deletions