<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>Hey Tom,</p>
<p>thank you for reporting this issue!<br>
</p>
<p>Tt looks like the check is confusing these overloads somehow.
Looking through the code it could be, that the second version is
considered as placement-overload which are ignored, but that is
just a guess.<br>
Are you by any chance compiling with C++17? It added versions to
both operators the check might not expect.<br>
</p>
<p>I created a bug-report:
<a class="moz-txt-link-freetext" href="https://bugs.llvm.org/show_bug.cgi?id=40174">https://bugs.llvm.org/show_bug.cgi?id=40174</a></p>
<p>You could help with producing a minimal reproducer-file with
C++-Standard and build-configuration added that we can attach to
the bug-report. Having such a file would help a lot adressing the
issue!<br>
</p>
<p>Best, Jonas</p>
<div class="moz-cite-prefix">Am 24.12.18 um 10:13 schrieb Tom
Hulton-Harrop via cfe-users:<br>
</div>
<blockquote type="cite"
cite="mid:CACxsqj0a4JCBFifeQrwtStaX5CtD3n=jiqOHEY8MiHgsW9Wt2w@mail.gmail.com">
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<div dir="auto">Hi there,
<div dir="auto"><br>
</div>
<div dir="auto">Apologies if this is the wrong place to ask
about this - if it is I'd be very grateful if you'd point me
in the right direction :)<br>
<div dir="auto"><br>
</div>
<div dir="auto">I'm seeing the clang-tidy warning
'misc-new-delete-overloads' when using the two parameter
version of class member delete, however when using the
single parameter version the warning goes away. </div>
<div dir="auto"><br>
</div>
<div dir="auto">class Type</div>
<div dir="auto">{</div>
<div dir="auto"> void* operator new(size_t) { ... }</div>
<div dir="auto"> void operator delete(void*) { ... } // no
warning</div>
<div dir="auto"> void operator delete (void*, size_t) { ...
} // warning</div>
<div dir="auto">};</div>
<div dir="auto"><br>
</div>
<div dir="auto">When using the second version I don't think
this warning should be reported. Is this correct or am I
missing something?</div>
<div dir="auto"><br>
</div>
<div dir="auto">I've been double checking things here:</div>
<div dir="auto"><br>
</div>
<div dir="auto"><a
href="https://en.cppreference.com/w/cpp/memory/new/operator_new"
moz-do-not-send="true">https://en.cppreference.com/w/cpp/memory/new/operator_new</a>
(Items 15, 16)<br>
</div>
<div dir="auto"><a
href="https://en.cppreference.com/w/cpp/memory/new/operator_delete"
moz-do-not-send="true">https://en.cppreference.com/w/cpp/memory/new/operator_delete</a>
(Items 17, 18 and 21, 22)<br>
</div>
<div dir="auto"><br>
</div>
<div dir="auto">I'm using clang on Windows and using Clang
Power Tools in Visual Studio to invoke clang-tidy.</div>
<div dir="auto"><br>
</div>
<div dir="auto">Any info would be greatly appreciated!</div>
<div dir="auto"><br>
</div>
<div dir="auto">Thank you for your time,</div>
<div dir="auto"><br>
</div>
<div dir="auto"><br>
</div>
<div dir="auto">Tom</div>
</div>
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<pre class="moz-quote-pre" wrap="">_______________________________________________
cfe-users mailing list
<a class="moz-txt-link-abbreviated" href="mailto:cfe-users@lists.llvm.org">cfe-users@lists.llvm.org</a>
<a class="moz-txt-link-freetext" href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-users">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-users</a>
</pre>
</blockquote>
</body>
</html>