You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by vi...@apache.org on 2017/12/06 19:04:48 UTC

mesos git commit: Removed redundant `get()` call on a smart pointer.

Repository: mesos
Updated Branches:
  refs/heads/master ecccb33ac -> dbaf85773


Removed redundant `get()` call on a smart pointer.

Registrar had the type `Owned<MockRegistry>` so this call was
indeed redundant.

Review: https://reviews.apache.org/r/64383/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/dbaf8577
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/dbaf8577
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/dbaf8577

Branch: refs/heads/master
Commit: dbaf857731735e40f5039538cc232cb236e25a0b
Parents: ecccb33
Author: Benno Evers <be...@mesosphere.com>
Authored: Wed Dec 6 11:04:24 2017 -0800
Committer: Vinod Kone <vi...@gmail.com>
Committed: Wed Dec 6 11:04:24 2017 -0800

----------------------------------------------------------------------
 src/tests/master_tests.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/dbaf8577/src/tests/master_tests.cpp
----------------------------------------------------------------------
diff --git a/src/tests/master_tests.cpp b/src/tests/master_tests.cpp
index 0dfa8c7..7b08767 100644
--- a/src/tests/master_tests.cpp
+++ b/src/tests/master_tests.cpp
@@ -2733,7 +2733,7 @@ TEST_F(MasterTest, RegistryUpdateAfterReconfiguration)
   // Verify master has correctly updated the registry.
   {
     Future<Response> response = process::http::get(
-        master.get()->registrar.get()->pid(),
+        master.get()->registrar->pid(),
         "registry",
         None(),
         createBasicAuthHeaders(DEFAULT_CREDENTIAL));
@@ -2866,7 +2866,7 @@ TEST_F(MasterTest, RegistryUpdateAfterMasterFailover)
   // Verify master has correctly updated the registry.
   {
     Future<Response> response = process::http::get(
-        master.get()->registrar.get()->pid(),
+        master.get()->registrar->pid(),
         "registry",
         None(),
         createBasicAuthHeaders(DEFAULT_CREDENTIAL));