Eric Blake <ebl...@redhat.com> writes: > On 08/08/2018 07:03 AM, Markus Armbruster wrote: >> Signed-off-by: Markus Armbruster <arm...@redhat.com> >> --- >> include/qapi/qmp/json-lexer.h | 10 ++++------ >> qobject/json-lexer.c | 18 ++++++++---------- >> 2 files changed, 12 insertions(+), 16 deletions(-) >> > >> @@ -335,8 +334,7 @@ static void json_lexer_feed_char(JSONLexer *lexer, char >> ch, bool flush) >> json_message_process_token(lexer, lexer->token, JSON_ERROR, >> lexer->x, lexer->y); >> g_string_truncate(lexer->token, 0); >> - new_state = lexer->start_state; >> - lexer->state = new_state; >> + lexer->state = lexer->start_state; > > Does this simplification belong in an earlier patch?
Hmm, PATCH 37 would be a better fit indeed. > Otherwise, > Reviewed-by: Eric Blake <ebl...@redhat.com> Thanks!
