You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@mesos.apache.org by Benno Evers <be...@mesosphere.com> on 2018/01/17 16:07:09 UTC

Re: Review Request 62444: Added UNREACHABLE() macro to __cxa_pure_virtual.

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

(Updated Jan. 17, 2018, 4:07 p.m.)


Review request for mesos, Benjamin Bannier and Till Toenshoff.


Changes
-------

Rebased onto latest master.


Summary (updated)
-----------------

Added UNREACHABLE() macro to __cxa_pure_virtual.


Repository: mesos


Description
-------

The function __cxa_pure_virtual must not return,
but newer versions of clang detect that the expansion
of the RAW_LOG() macro contains returning code paths
for arguments other than FATAL.


Diffs (updated)
-----

  src/logging/logging.cpp 19dba7faac7f2307e5dfdf15f64849f643b19129 


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

Changes: https://reviews.apache.org/r/62444/diff/1-2/


Testing
-------


Thanks,

Benno Evers