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?
Otherwise,
Reviewed-by: Eric Blake <ebl...@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
