Richard Heck wrote: > commit b684ec39eaaf7d5b0270bf8109b8e281fa8c5d9f > Author: Richard Heck <rgh...@lyx.org> > Date: Fri Dec 15 23:50:23 2017 -0500 > > Fix #10863 compiler warnings. > --- > src/mathed/MathParser.cpp | 12 +++++------- > 1 files changed, 5 insertions(+), 7 deletions(-) > > diff --git a/src/mathed/MathParser.cpp b/src/mathed/MathParser.cpp > index 35062ca..c914a6a 100644 > --- a/src/mathed/MathParser.cpp > +++ b/src/mathed/MathParser.cpp > @@ -744,7 +744,6 @@ docstring Parser::parse_verbatim_option() > skipSpaces(); > docstring res; > if (nextToken().character() == '[') { > - Token t = getToken(); > for (Token t = getToken(); t.character() != ']' && good(); t = > getToken()) { > if (t.cat() == catBegin) { > putback(); > @@ -762,7 +761,6 @@ docstring Parser::parse_verbatim_item() > skipSpaces(); > docstring res; > if (nextToken().cat() == catBegin) { > - Token t = getToken(); > for (Token t = getToken(); t.cat() != catEnd && good(); t = > getToken()) { > if (t.cat() == catBegin) { > putback();
I don't see wider context, but it seems the new codes has one less getToken() call. Doesn't it have some side effect? P