For CX_BinaryOperatorKind please change the enumerators to have an explicit 
integer assignment; this is to emphasize that they are API contract and should 
not change with subsequent changes (someone may inadvertently add a new 
enumerator in the middle which will implicitly change all the subsequent 
enumerators).

Otherwise LGTM.

> On Dec 6, 2015, at 11:45 PM, guibufolo+l...@gmail.com wrote:
> 
> RedX2501 added a comment.
> 
> Ping
> 
> 
> http://reviews.llvm.org/D10833
> 
> 
> 

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

Reply via email to