diff options
author | Christian Kandeler <christian.kandeler@qt.io> | 2016-12-15 14:05:07 +0100 |
---|---|---|
committer | Christian Kandeler <christian.kandeler@qt.io> | 2016-12-16 10:16:40 +0000 |
commit | 7ebdd6ef8002e949469ed4dad4be8414dd0b2aab (patch) | |
tree | 4811ecfdafe0627bd6b7318fe43910ab409b46b6 /src/lib/corelib/language/artifactproperties.cpp | |
parent | f0b106759521ad64772b89047981b99c53926bd7 (diff) |
PersistentPool: Prevent access to raw data stream
Instead, we employ our now-generic load/store interface for all
serialization operations. The advantages are two-fold:
1) It can no longer happen that carelessly written store()
functions mistakenly dump unshared strings into the build graph,
as it has happened several times in the past.
2) We get rid of enormous amounts of custom code dealing with
container (de)serialization. All of this is now done centrally by a
handful of templates in PersistentPool.
Change-Id: Ib1262da50989edda9726abd7d5a8ffdceaa0ec76
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
Diffstat (limited to 'src/lib/corelib/language/artifactproperties.cpp')
-rw-r--r-- | src/lib/corelib/language/artifactproperties.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/lib/corelib/language/artifactproperties.cpp b/src/lib/corelib/language/artifactproperties.cpp index 80054ff2d..8cb1770dc 100644 --- a/src/lib/corelib/language/artifactproperties.cpp +++ b/src/lib/corelib/language/artifactproperties.cpp @@ -55,13 +55,13 @@ ArtifactProperties::ArtifactProperties() void ArtifactProperties::load(PersistentPool &pool) { - m_fileTagsFilter.load(pool); + pool.load(m_fileTagsFilter); pool.load(m_propertyMap); } void ArtifactProperties::store(PersistentPool &pool) const { - m_fileTagsFilter.store(pool); + pool.store(m_fileTagsFilter); pool.store(m_propertyMap); } |