Dead assignment deletion

[Ted Kremenek]

Thank you!  Committed in r202115.

On Feb 23, 2014, at 2:58 PM, Gautier DI FOLCO <gautier.difolco at gmail.com> wrote:

> Hello,
> 
> I wanted to contribute to some FOSS projects and when I followed Sylvestre LEDRU's talk during FOSDEM, it has appeared to me that there were some simple tasks to do on Clang, so it would be a good starting point.
> 
> 
> So I'm gone on http://buildd-clang.debian.net/scan-build/, looking for some dead assignments to delete.
> I have found one in clang /lib /Sema /SemaObjCProperty.cpp on line 1819.
> I have checkout the projects (at revision 201990), built them once, I have ran the clang's test suite, which give me :
> 
> gmake[1]: Entering directory `/usr/home/black/build/tools/clang/test'
> Making Clang 'lit.site.cfg' file...
> Making Clang 'Unit/lit.site.cfg' file...
> --- Running clang tests for x86_64-unknown-freebsd9.2 ---
> lit.py: lit.cfg:196: note: using clang: '/usr/home/black/build/Debug+Asserts/bin/clang'
> Testing Time: 313.95s
>   Expected Passes    : 6939
>   Expected Failures  : 20
>   Unsupported Tests  : 15
> gmake[1]: Leaving directory `/usr/home/black/build/tools/clang/test'
> 
> I have then dropped the line, rebuilt the projects, I have relaunched tests and I have had the same results.
> 
> I don't know if I follow the rules, I'm sorry if I don't and I will take any links which can explains me how to follow them.
> 
> It is not a big contribution but I want to do many small ones to immerse myself into this projet. If you think that there is easy tasks more useful to do, let me know. If this first contribution is a success, I have planned to drop the majority of bugs in this list, so do you think it is wise to do one patch for each "line" or a bigger patch which will aggregate 10 of them?
> 
> Thank you for your time,
> Regards.
> 
> <dead-assignment.patch>_______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140224/6c9f4d58/attachment.html>