aaron.ballman added a comment.

In D31130#2220729 <https://reviews.llvm.org/D31130#2220729>, @fiesh wrote:

> Ping everybody?  This is a rather important issue I think since it makes 
> clang-tidy not usable in a lot of cases, and the fix would be all done.  
> Please let's get this merged!

Sorry for missing this before -- it's totally fine to pick up someone else's 
patch if the original author isn't able to work on it. IANAL, so I can't 
comment on the licensing question with any authority, but given that the 
original author says they're fine giving up full rights to their patch, that 
seems sufficient to me.

Given how long this review has sat idle for, I'd recommend starting a new 
review with a fully rebased patch, though.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D31130/new/

https://reviews.llvm.org/D31130

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to