[PATCH] D12047: test/SemaObjC: Remove cruft in unused getter test

Alexey Denisov via cfe-commits cfe-commits at lists.llvm.org
Sat Aug 15 06:39:13 PDT 2015


AlexDenisov added a comment.

s/which is important/which is might be important'/ 
;)

IMHO, the purpose of the test is not just prove that functionality is working as expected, but also prevent from regression, I completely agree that this test can be cleaned up a bit, but I'd cover at least two cases: a property of a class itself and a property of a superclass.
Though, it's better to ask somebody with a proper understanding of internals :)


http://reviews.llvm.org/D12047





More information about the cfe-commits mailing list