mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Benjamin Bannier <>
Subject Review Request 46084: Added needed forward-declaration for InternalServerError.
Date Tue, 12 Apr 2016 13:06:14 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for mesos, Adam B, Alexander Rojas, and Joerg Schad.

Bugs: MESOS-5152

Repository: mesos


As currently written the code relies on `process::http` being the only
visible namespace with name `http`, an assumption which will break as
soon as we introduce declarations from `mesos::http`.

Fix this as customary by explicitly pulling the needed name into the
local name lookup scope.


  src/tests/slave_tests.cpp 4a576b98d1cc58072626ac2c41c599bd3c8385c5 



make check (clang, w/o optimization, OS X)

Without this patch including e.g., `mesos/authentication/http/basic_authenticator_factory.hpp`
leads to a compilation failure.


Benjamin Bannier

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