[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 05/10] json-lexer: Drop 'buf'
From: |
Luiz Capitulino |
Subject: |
[Qemu-devel] [PATCH 05/10] json-lexer: Drop 'buf' |
Date: |
Fri, 11 Jun 2010 16:58:10 -0300 |
QString supports adding a single char, 'buf' is unneeded.
Signed-off-by: Luiz Capitulino <address@hidden>
---
json-lexer.c | 7 +------
1 files changed, 1 insertions(+), 6 deletions(-)
diff --git a/json-lexer.c b/json-lexer.c
index 5cc7e6c..1d9b81f 100644
--- a/json-lexer.c
+++ b/json-lexer.c
@@ -284,8 +284,6 @@ void json_lexer_init(JSONLexer *lexer, JSONLexerEmitter
func)
static int json_lexer_feed_char(JSONLexer *lexer, char ch)
{
- char buf[2];
-
lexer->x++;
if (ch == '\n') {
lexer->x = 0;
@@ -313,10 +311,7 @@ static int json_lexer_feed_char(JSONLexer *lexer, char ch)
break;
}
- buf[0] = ch;
- buf[1] = 0;
-
- qstring_append(lexer->token, buf);
+ qstring_append_chr(lexer->token, ch);
return 0;
}
--
1.7.1.251.g92a7
- [Qemu-devel] [PATCH 00/10][PULL]: QMP/Monitor queue, Luiz Capitulino, 2010/06/11
- [Qemu-devel] [PATCH 01/10] json-lexer: Initialize 'x' and 'y', Luiz Capitulino, 2010/06/11
- [Qemu-devel] [PATCH 02/10] json-lexer: Handle missing escapes, Luiz Capitulino, 2010/06/11
- [Qemu-devel] [PATCH 04/10] check-qjson: Add more escape tests, Luiz Capitulino, 2010/06/11
- [Qemu-devel] [PATCH 03/10] qjson: Handle "\f", Luiz Capitulino, 2010/06/11
- [Qemu-devel] [PATCH 05/10] json-lexer: Drop 'buf',
Luiz Capitulino <=
- [Qemu-devel] [PATCH 06/10] json-streamer: Don't use qdict_put_obj(), Luiz Capitulino, 2010/06/11
- [Qemu-devel] [PATCH 08/10] implement optional lookahead in json lexer, Luiz Capitulino, 2010/06/11
- [Qemu-devel] [PATCH 07/10] add some tests for invalid JSON, Luiz Capitulino, 2010/06/11
- [Qemu-devel] [PATCH 09/10] remove unnecessary lookaheads, Luiz Capitulino, 2010/06/11
- [Qemu-devel] [PATCH 10/10] hxtool: Fix line number reporting on SQMP/EQMP errors, Luiz Capitulino, 2010/06/11
- Re: [Qemu-devel] [PATCH 00/10][PULL]: QMP/Monitor queue, Anthony Liguori, 2010/06/14