This is an automated email from the ASF dual-hosted git repository.

bbannier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mesos.git

commit d63be44d94c5763a26c97990ac4236d60a4c2c18
Author: Benjamin Bannier <[email protected]>
AuthorDate: Wed Aug 14 09:33:14 2019 +0200

    Fixed signature to pass parameter by const ref instead of value.
    
    Review: https://reviews.apache.org/r/71145/
---
 src/tests/storage_local_resource_provider_tests.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/tests/storage_local_resource_provider_tests.cpp 
b/src/tests/storage_local_resource_provider_tests.cpp
index 69b59d4..57b0c55 100644
--- a/src/tests/storage_local_resource_provider_tests.cpp
+++ b/src/tests/storage_local_resource_provider_tests.cpp
@@ -243,10 +243,10 @@ public:
 
   void setupResourceProviderConfig(
       const Bytes& capacity,
-      const Option<string> volumes = None(),
-      const Option<string> forward = None(),
-      const Option<string> createParameters = None(),
-      const Option<string> volumeMetadata = None())
+      const Option<string>& volumes = None(),
+      const Option<string>& forward = None(),
+      const Option<string>& createParameters = None(),
+      const Option<string>& volumeMetadata = None())
   {
     const string testCsiPluginPath =
       path::join(tests::flags.build_dir, "src", "test-csi-plugin");

Reply via email to