<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jun 20, 2016 at 8:17 AM, Ben Boeckel <span dir="ltr"><<a href="mailto:ben.boeckel@kitware.com" target="_blank">ben.boeckel@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><br>
and our copy already has it:<br>
<br>
    <a href="https://gitlab.kitware.com/third-party/jsoncpp/blob/master/src/lib_json/json_reader.cpp#L46" rel="noreferrer" target="_blank">https://gitlab.kitware.com/third-party/jsoncpp/blob/master/src/lib_json/json_reader.cpp#L46</a></blockquote><div><br></div><div>I'm fond of simple solutions like that.  Since we know it works, we could add the following to <a href="http://vtkConfigure.h.in">vtkConfigure.h.in</a>, which is easier than adding a try-compile:</div><div><br></div><div>#if __cplusplus >= 201103L || (defined(_CPPLIB_VER) && _CPPLIB_VER >= 520)</div><div>#define VTK_HAS_STD_UNIQUE_PTR</div><div>#endif</div><div><br></div><div>Those three source files can use this in order to avoid the compiler warning (i.e. by using std::unique_ptr when available).</div><div><br></div><div>I don't think we should actively encourage the use of std::unique_ptr yet.  That can wait until we're ready to abandon the old compilers.</div><div><br></div><div> - David</div></div></div></div>