[PATCH] D43120: [clang-tidy] New checker for exceptions that are created but not thrown

Roman Lebedev via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 15 04:58:25 PST 2018


On Thu, Feb 15, 2018 at 3:48 PM, Aaron Ballman via cfe-commits
<cfe-commits at lists.llvm.org> wrote:
> On Thu, Feb 15, 2018 at 4:11 AM, Phabricator via Phabricator
> <reviews at reviews.llvm.org> wrote:
>> This revision was automatically updated to reflect the committed changes.
>> Closed by commit rL325222: [clang-tidy] New checker for exceptions that are created but not thrown (authored by xazax, committed by ).
>
> There's nothing wrong with this patch, but it is odd that the
> automated text describes the author but not the committer. I wonder if
> that's a bug?

Committed xazax Thu, Feb 15, 12:08
But there is no https://reviews.llvm.org/p/xazax/, only
https://reviews.llvm.org/p/xazax.hun/

I get that too.
Because committer name (i.e. svn username) does not verbatim match the
Phab username verbatim.

It is a bug i'd say, the match should be email-based.

> ~Aaron
Roman.

> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


More information about the llvm-commits mailing list