You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Tomasz Janiszewski (JIRA)" <ji...@apache.org> on 2016/03/28 23:01:25 UTC

[jira] [Issue Comment Deleted] (MESOS-3243) Replace NULL with nullptr

     [ https://issues.apache.org/jira/browse/MESOS-3243?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tomasz Janiszewski updated MESOS-3243:
--------------------------------------
    Comment: was deleted

(was: I fixed PR. And made two separate commits ([diff|https://github.com/apache/mesos/compare/077ca9505c533f1c8fcf94e3cc1779d873ab8bea...c4ef738dbdf92fcaaa8724a8246d6f9c7bb85f7f])

1. {{clang-tidy}} [077ca9505c533f1c8fcf94e3cc1779d873ab8bea|https://github.com/apache/mesos/commit/077ca9505c533f1c8fcf94e3cc1779d873ab8bea]
2. {{sed}} [c4ef738dbdf92fcaaa8724a8246d6f9c7bb85f7f|https://github.com/apache/mesos/commit/c4ef738dbdf92fcaaa8724a8246d6f9c7bb85f7f]



Should I open new review and post them one by one?)

> Replace NULL with nullptr
> -------------------------
>
>                 Key: MESOS-3243
>                 URL: https://issues.apache.org/jira/browse/MESOS-3243
>             Project: Mesos
>          Issue Type: Improvement
>            Reporter: Michael Park
>            Assignee: Tomasz Janiszewski
>
> As part of the C++ upgrade, it would be nice to move our use of {{NULL}} over to use {{nullptr}}. I think it would be an interesting exercise to do this with {{clang-modernize}} using the [nullptr transform|http://clang.llvm.org/extra/UseNullptrTransform.html] (although it's probably just as easy to use {{sed}}).



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