[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