Re: [clang-tools-extra] r284894 - [Release notes] Mention removed Clang-tidy misc-pointer-and-integral-operation check

[Eugene Zelenko via cfe-commits]

Hi, Kim!

Sure, it'll be good idea, but I think, it'll be better, if Richard
Smith will do this. I just followed r284888, which doesn't contain a
lot of explanations or reference to relevant discussion.

Eugene.

On Sat, Oct 22, 2016 at 12:48 AM, Kim Gräsman  wrote:
> Hi Eugene,
>
> On Sat, Oct 22, 2016 at 12:35 AM, Eugene Zelenko via cfe-commits
>  wrote:
>> Author: eugenezelenko
>> Date: Fri Oct 21 17:35:58 2016
>> New Revision: 284894
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=284894=rev
>> Log:
>> [Release notes] Mention removed Clang-tidy 
>> misc-pointer-and-integral-operation check
>>
>> Modified:
>> clang-tools-extra/trunk/docs/ReleaseNotes.rst
>>
>> Modified: clang-tools-extra/trunk/docs/ReleaseNotes.rst
>> URL: 
>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/ReleaseNotes.rst?rev=284894=284893=284894=diff
>> ==
>> --- clang-tools-extra/trunk/docs/ReleaseNotes.rst (original)
>> +++ clang-tools-extra/trunk/docs/ReleaseNotes.rst Fri Oct 21 17:35:58 2016
>> @@ -73,6 +73,8 @@ Improvements to clang-tidy
>>Warns when ``std::move`` is applied to a forwarding reference instead of
>>``std::forward``.
>>
>> +- `misc-pointer-and-integral-operation` check was removed.
>> +
>
> Would it be worthwhile to describe *why* it was removed? I haven't
> followed the discussions, so this looks a little abrupt.
>
> - Kim
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [clang-tools-extra] r284894 - [Release notes] Mention removed Clang-tidy misc-pointer-and-integral-operation check

[Kim Gräsman via cfe-commits]

Hi Eugene,

On Sat, Oct 22, 2016 at 12:35 AM, Eugene Zelenko via cfe-commits
 wrote:
> Author: eugenezelenko
> Date: Fri Oct 21 17:35:58 2016
> New Revision: 284894
>
> URL: http://llvm.org/viewvc/llvm-project?rev=284894=rev
> Log:
> [Release notes] Mention removed Clang-tidy 
> misc-pointer-and-integral-operation check
>
> Modified:
> clang-tools-extra/trunk/docs/ReleaseNotes.rst
>
> Modified: clang-tools-extra/trunk/docs/ReleaseNotes.rst
> URL: 
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/docs/ReleaseNotes.rst?rev=284894=284893=284894=diff
> ==
> --- clang-tools-extra/trunk/docs/ReleaseNotes.rst (original)
> +++ clang-tools-extra/trunk/docs/ReleaseNotes.rst Fri Oct 21 17:35:58 2016
> @@ -73,6 +73,8 @@ Improvements to clang-tidy
>Warns when ``std::move`` is applied to a forwarding reference instead of
>``std::forward``.
>
> +- `misc-pointer-and-integral-operation` check was removed.
> +

Would it be worthwhile to describe *why* it was removed? I haven't
followed the discussions, so this looks a little abrupt.

- Kim
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits