[cfe-commits] r163329 - in /cfe/trunk: lib/Sema/SemaExpr.cpp test/Sema/warn-unused-function.c

[jahanian]

On Sep 6, 2012, at 10:45 AM, Ted Kremenek wrote:

> On Sep 6, 2012, at 10:33 AM, jahanian <fjahanian at apple.com> wrote:
> 
>> On Sep 6, 2012, at 10:28 AM, Ted Kremenek wrote:
>> 
>>> Hi Fariborz,
>>> 
>>> I see logic here that special cases for Objective-C categories, but no test case for that bit.  Is that already covered by an existing test?
>> No, I cut-and-pasted from similar code for other attributes. I will add a test case.
> 
> Instead of copy-and-pasting, can we refactor the common logic to go through a common helper function?

It is a two line of code. I am not sure that it is worse it.

- fariborz


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120906/9c37eb65/attachment.html>