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

Posted to issues@mesos.apache.org by "Benjamin Bannier (JIRA)" <ji...@apache.org> on 2016/12/21 15:57:58 UTC

[jira] [Commented] (MESOS-6824) mesos-this-capture clang-tidy check has false positives

    [ https://issues.apache.org/jira/browse/MESOS-6824?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15767396#comment-15767396 ] 

Benjamin Bannier commented on MESOS-6824:
-----------------------------------------

[~mcypark], could you help with this?

> mesos-this-capture clang-tidy check has false positives
> -------------------------------------------------------
>
>                 Key: MESOS-6824
>                 URL: https://issues.apache.org/jira/browse/MESOS-6824
>             Project: Mesos
>          Issue Type: Bug
>            Reporter: Benjamin Bannier
>            Assignee: Benjamin Bannier
>              Labels: clang-tidy
>
> The {{mesos-this-capture}} clang-tidy checks incorrectly triggers on the code here,
>   https://github.com/apache/mesos/blob/d2117362349ab4c383045720f77d42b2d9fd6871/src/slave/containerizer/mesos/io/switchboard.cpp#L1487
> We should tighten the matcher to avoid triggering on such constructs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)