summaryrefslogtreecommitdiff
path: root/modules/pbm/bitmap_loader_pbm.cpp
diff options
context:
space:
mode:
authorRĂ©mi Verschelde <rverschelde@gmail.com>2017-08-22 00:58:12 +0200
committerGitHub <noreply@github.com>2017-08-22 00:58:12 +0200
commitdf590fc2d33c1ba715a4ce58f71d83e0ed9f5693 (patch)
tree8bfdc3e033aaf4dc9a5de5e8040e4f99afa0c540 /modules/pbm/bitmap_loader_pbm.cpp
parent13f879587dd9bce59528e44b0faaf6e062f6d918 (diff)
parent738d2ab96997faa1e13b91e38cf8a0000d829f70 (diff)
Merge pull request #10340 from Rubonnek/remove-unnecessary-assignments
Removed unnecessary assignments
Diffstat (limited to 'modules/pbm/bitmap_loader_pbm.cpp')
-rw-r--r--modules/pbm/bitmap_loader_pbm.cpp3
1 files changed, 3 insertions, 0 deletions
diff --git a/modules/pbm/bitmap_loader_pbm.cpp b/modules/pbm/bitmap_loader_pbm.cpp
index c8f25ad68c..93dedbd05f 100644
--- a/modules/pbm/bitmap_loader_pbm.cpp
+++ b/modules/pbm/bitmap_loader_pbm.cpp
@@ -95,6 +95,9 @@ static bool _get_token(FileAccessRef &f, uint8_t &saved, PoolVector<uint8_t> &r_
resized = true;
}
if (resized) {
+ // Note: Certain C++ static analyzers might point out that the following assigment is unnecessary.
+ // This is wrong since PoolVector<class T>::Write has an operator= method where the lhs gets updated under certain conditions.
+ // See core/dvector.h.
w = PoolVector<uint8_t>::Write();
r_token.resize(token_max);
w = r_token.write();