mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <benjamin.bann...@mesosphere.io>
Subject Re: Review Request 65161: Removed some redundant `get` calls in libprocess.
Date Wed, 21 Feb 2018 13:31:17 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65161/
-----------------------------------------------------------

(Updated Feb. 21, 2018, 2:31 p.m.)


Review request for mesos, Andrew Schwartzmeyer and Michael Park.


Repository: mesos


Description
-------

Removed some redundant `get` calls in libprocess.


Diffs
-----

  3rdparty/libprocess/include/process/gmock.hpp 231efcd989b829b5cfba8e5d69ca5cb2c4bf0428 
  3rdparty/libprocess/include/process/gtest.hpp eee726653d52af2e4a148819e420ebd22e5123a9 
  3rdparty/libprocess/include/process/system.hpp 21bd3300b104eaa56642f19c3dcb95950ab94830

  3rdparty/libprocess/src/http.cpp 76a65e02fd83219b198d0b8fde3f8208e1c606b5 
  3rdparty/libprocess/src/http_proxy.cpp f584238aadd86875d7c87736653f394e716397de 
  3rdparty/libprocess/src/metrics/metrics.cpp f79541853b4c826014ee969633345c3d51520ecf 
  3rdparty/libprocess/src/openssl.cpp f2fe90bfdf87dbc674b231f6392e95a95ebd9736 
  3rdparty/libprocess/src/process.cpp 42e7adf740b234ebf23d2dcb71440749c0ed87ec 
  3rdparty/libprocess/src/ssl/utilities.cpp 9b1d6229c3aa7d7045d492d377223d56bca16156 
  3rdparty/libprocess/src/subprocess_posix.hpp 25eb44ab7f68907c30008ae729403a986cb1898a 
  3rdparty/libprocess/src/tests/benchmarks.cpp 48b5f5cf38d8fb4d77025f9723031075ca8ab677 
  3rdparty/libprocess/src/tests/collect_tests.cpp 7c2ba902d2fba340a5fae49a911a9e84223fe1ce

  3rdparty/libprocess/src/tests/future_tests.cpp bd98154cfa2020b7423951f070fca7b0447d3cd5

  3rdparty/libprocess/src/tests/http_tests.cpp a60f6b132aff00da42e733f5c5f387dff2428ed2 
  3rdparty/libprocess/src/tests/metrics_tests.cpp 5a5a2e581d0917ad5aa6aa93257ec0dddc895725

  3rdparty/libprocess/src/tests/process_tests.cpp 96b1675dea7df675667c61ff2bd1d1cc8231a457

  3rdparty/libprocess/src/tests/reap_tests.cpp f8a8593b685c5e5a9321d970a57e9ebdff127c24 
  3rdparty/libprocess/src/tests/ssl_tests.cpp 71f2df8cd2377ae1be762ec7a0fb0d2f7b210e4b 
  3rdparty/libprocess/src/tests/statistics_tests.cpp 144b5109cfb7640b29bec8de8f5b2ad00665212f

  3rdparty/libprocess/src/tests/subprocess_tests.cpp 8211ae63e75359705af2bb9ac02a7cbdb28e3252

  3rdparty/libprocess/src/tests/system_tests.cpp 6beb973e079c00ba3e632d6fb6c3c332483d203f

  3rdparty/libprocess/src/tests/timeseries_tests.cpp 4177fb673bbf510fe835af396cd76dae48a9ec85



Diff: https://reviews.apache.org/r/65161/diff/2/


Testing (updated)
-------

* `make check`
* tested on a number of platforms and configurations on internal CI

These changes were made automatically with https://github.com/mesos/clang-tools-extra/pull/13
with only a few minor additions in surrounding code.


Thanks,

Benjamin Bannier


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message