diff options
author | Thiago Marcos P. Santos <tmpsantos@gmail.com> | 2020-03-27 18:30:35 +0200 |
---|---|---|
committer | Thiago Marcos P. Santos <tmpsantos@gmail.com> | 2020-04-17 13:36:50 +0300 |
commit | b49517b73705e69d401925658ae0657b8a98b6a6 (patch) | |
tree | 617a0f7493b0071be01e272efc819f15154190e7 /include/mbgl | |
parent | 542c11ae63f4afff2e0c84712019c0f9f19ef590 (diff) | |
download | qtlocation-mapboxgl-b49517b73705e69d401925658ae0657b8a98b6a6.tar.gz |
[core] Fix performance-unnecessary-value-param errors in header files
As reported by clang-tidy-8.
Diffstat (limited to 'include/mbgl')
-rw-r--r-- | include/mbgl/renderer/renderer_observer.hpp | 2 | ||||
-rw-r--r-- | include/mbgl/storage/file_source.hpp | 4 | ||||
-rw-r--r-- | include/mbgl/storage/offline.hpp | 2 |
3 files changed, 5 insertions, 3 deletions
diff --git a/include/mbgl/renderer/renderer_observer.hpp b/include/mbgl/renderer/renderer_observer.hpp index e0fc84215e..5d5a7bb5fe 100644 --- a/include/mbgl/renderer/renderer_observer.hpp +++ b/include/mbgl/renderer/renderer_observer.hpp @@ -34,7 +34,7 @@ public: // Style is missing an image using StyleImageMissingCallback = std::function<void()>; - virtual void onStyleImageMissing(const std::string&, StyleImageMissingCallback done) { done(); } + virtual void onStyleImageMissing(const std::string&, const StyleImageMissingCallback& done) { done(); } virtual void onRemoveUnusedStyleImages(const std::vector<std::string>&) {} }; diff --git a/include/mbgl/storage/file_source.hpp b/include/mbgl/storage/file_source.hpp index 0ce88981a5..562fd8da08 100644 --- a/include/mbgl/storage/file_source.hpp +++ b/include/mbgl/storage/file_source.hpp @@ -48,6 +48,8 @@ public: // Allows to forward response from one source to another. // Optionally, callback can be provided to receive notification for forward // operation. + // + // NOLINTNEXTLINE(performance-unnecessary-value-param) virtual void forward(const Resource&, const Response&, std::function<void()>) {} // When a file source supports consulting a local cache only, it must return true. @@ -82,7 +84,7 @@ public: virtual mapbox::base::Value getProperty(const std::string&) const { return {}; }; // When supported, sets the modifier of the requested resources. - virtual void setResourceTransform(ResourceTransform) {} + virtual void setResourceTransform(ResourceTransform) {} // NOLINT(performance-unnecessary-value-param) protected: FileSource() = default; diff --git a/include/mbgl/storage/offline.hpp b/include/mbgl/storage/offline.hpp index f35abbfe38..97e7fc2e49 100644 --- a/include/mbgl/storage/offline.hpp +++ b/include/mbgl/storage/offline.hpp @@ -191,7 +191,7 @@ public: * responsibility of the SDK bindings to wrap this object in an interface that * re-executes the user-provided implementation on the main thread. */ - virtual void responseError(Response::Error) {} + virtual void responseError(Response::Error) {} // NOLINT(performance-unnecessary-value-param) /* * Implement this method to be notified when the limit on the number of Mapbox |