On Wed, Apr 13, 2011 at 2:02 AM, <cr...@google.com> wrote: > One oddity, otherwise LGTM. > > > http://codereview.appspot.com/4389045/diff/1/gcc/cp/parser.c > File gcc/cp/parser.c (right): > > http://codereview.appspot.com/4389045/diff/1/gcc/cp/parser.c#newcode375 > gcc/cp/parser.c:375: if (flag) > This code will never print "false", so why have it?
Can you quote the piece of the patch you are commenting on? Thanks, Richard. > http://codereview.appspot.com/4389045/ >