[PATCH] D22069: clang-tidy modernize-loop-convert: preserve type of alias declaration (bug 28341)

[Alexander Kornienko via cfe-commits]

alexfh added a comment.

Thank you for fixing the bug! One comment below.


================
Comment at: clang-tidy/modernize/LoopConvertCheck.cpp:525
@@ +524,3 @@
+      DeclarationType = DeclarationType.getNonReferenceType();
+    if (Descriptor.ElemType.isNull() || DeclarationType.isNull() ||
+        !Context->hasSameUnqualifiedType(DeclarationType, Descriptor.ElemType))
----------------
1. Can the `AliasVar->getType().isNull()` condition be true?
2. If it can, consider `!Descriptor.ElemType.isNull().isNull()` and `AliasVar->getType().isNull()`. In this case setting `Descriptor.ElemType` to `AliasVar->getType()` (which is null) doesn't make much sense.

I'd probably just wrap the added code in `if (!AliasVar->getType().isNull())`.


http://reviews.llvm.org/D22069