<div dir="ltr">I have merged this as a new large commit. I'll try fixing any issues that crop up on the nightlies tomorrow. There are still many uses of NULL in comments and classes that did not build on my linux system (such as Win32 and OSX specific classes) If you see uses of NULL or 0 in code you are editing that should be nullptr, please update them.<div><br></div><div>I did go beyond just clang-tidy so if you see issues/regressions with the change let me know. <br></div><div><br></div><div>Going forward in C++ code please use nullptr as it has better type safety and overload resolution than NULL or 0. </div><div><br></div><div>In comments I tried to use "nullptr" or "null" depending on the context as opposed to NULL.</div><div><br></div><div>Topics in flight may have merge conflicts (probably :-( ).</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 25, 2017 at 2:08 PM, Ken Martin <span dir="ltr"><<a href="mailto:ken.martin@kitware.com" target="_blank">ken.martin@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>Here is a topic for updating VTK/Common to use nullptr based on fixes from clang-tidy</div><div><br></div><div><a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/3055" target="_blank">https://gitlab.kitware.com/<wbr>vtk/vtk/merge_requests/3055</a><br></div><div><br></div><div>I just did common as I figured that would impact fewer people. My plan is to do other directories in chunks over time but I can go whole hog and do it all at once if you all think that is better.  The change is from Rob's script ala...</div><div><br></div><div><div>#!/bin/bash</div><div><br></div><div># The vtk build must have CMAKE_EXPORT_COMPILE_COMMANDS=<wbr>ON</div><div><br></div><div>path=$(pwd)</div><div>build_path="/home/ken/<wbr>Documents/vtk/vtkbin"</div><div>clang_tidy="/usr/bin/clang-<wbr>tidy"</div><div>args="-checks=-*,modernize-<wbr>use-nullptr --header-filter=$path -p $build_path -fix"</div><div>for file in $1/*.cxx; do<br></div><div>  $clang_tidy $args $file</div><div>done</div></div><span class="HOEnZb"><font color="#888888"><div><br></div><br clear="all"><div><br></div>-- <br><div class="m_-4532524404558867768gmail_signature"><div dir="ltr"><div>Ken Martin PhD<div>Distinguished Engineer<br><span style="font-size:12.8px">Kitware Inc.</span><br></div><div>28 Corporate Drive<br>Clifton Park NY 12065<br><div><br></div><div><span style="font-size:10pt;font-family:Tahoma,sans-serif">This communication,
including all attachments, contains confidential and legally privileged
information, and it is intended only for the use of the addressee.  Access to this email by anyone else is
unauthorized. If you are not the intended recipient, any disclosure, copying,
distribution or any action taken in reliance on it is prohibited and may be
unlawful. If you received this communication in error please notify us
immediately and destroy the original message. 
Thank you.</span></div></div></div></div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Ken Martin PhD<div>Distinguished Engineer<br><span style="font-size:12.8px">Kitware Inc.</span><br></div><div>28 Corporate Drive<br>Clifton Park NY 12065<br><div><br></div><div><span style="font-size:10pt;font-family:Tahoma,sans-serif">This communication,
including all attachments, contains confidential and legally privileged
information, and it is intended only for the use of the addressee.  Access to this email by anyone else is
unauthorized. If you are not the intended recipient, any disclosure, copying,
distribution or any action taken in reliance on it is prohibited and may be
unlawful. If you received this communication in error please notify us
immediately and destroy the original message. 
Thank you.</span></div></div></div></div></div>
</div>