[cfe-commits] r148212 - /cfe/trunk/lib/Lex/Lexer.cpp

Chandler Carruth chandlerc at gmail.com
Sun Jan 15 20:35:48 PST 2012


On Sun, Jan 15, 2012 at 5:23 PM, David Blaikie <dblaikie at gmail.com> wrote:

> On Sun, Jan 15, 2012 at 1:03 AM, Chandler Carruth <chandlerc at gmail.com>
> wrote:
> > Author: chandlerc
> > Date: Sun Jan 15 03:03:45 2012
> > New Revision: 148212
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=148212&view=rev
> > Log:
> > Two variables had been added for an assert, but their values were
> > re-computed rather than the variables be re-used just after the assert.
> > Just use the variables since we have them already. Fixes an unused
> > variable warning.
>
> Thanks Chandler - attached are some patches for a few other release
> unused variable warnings if you think they're the right fixes.


Yes, these are fine. Feel free to commit, and similar obvious
simplifications which reduce warnings.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120115/2d55b0dc/attachment.html>


More information about the cfe-commits mailing list