mesos-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yong Tang (JIRA)" <>
Subject [jira] [Commented] (MESOS-4602) Invalid usage of ATOMIC_FLAG_INIT in member initialization
Date Wed, 24 Feb 2016 17:48:18 GMT


Yong Tang commented on MESOS-4602:

The patch has been applied. Thanks [~bbannier] and  [~tillt] for reviews.

> Invalid usage of ATOMIC_FLAG_INIT in member initialization
> ----------------------------------------------------------
>                 Key: MESOS-4602
>                 URL:
>             Project: Mesos
>          Issue Type: Bug
>          Components: libprocess
>            Reporter: Benjamin Bannier
>            Assignee: Yong Tang
>              Labels: newbie, tech-debt
> MESOS-2925 fixed a few instances where {{ATOMIC_FLAG_INIT}} was used in initializer lists,
but missed to fix {{3rdparty/libprocess/src/libevent_ssl_socket.cpp}} (even though the corresponding
header was touched).
> There, {{LibeventSSLSocketImpl}}'s {{lock}} member is still (incorrectly) initialized
in initializer lists, even though the member is already initialized in the class declaration,
so it appears they should be dropped.
> Clang from trunk incorrectly diagnoses the initializations in the initializer lists as
benign redundant braces in initialization of a scalar, but they should be fixed for the reasons
stated in MESOS-2925.

This message was sent by Atlassian JIRA

View raw message