Eric Blake <ebl...@redhat.com> writes: > On 01/16/2013 07:36 AM, Markus Armbruster wrote: >> Signed-off-by: Markus Armbruster <arm...@redhat.com> >> --- >> readline.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/readline.c b/readline.c >> index a0c9638..d6e04d4 100644 >> --- a/readline.c >> +++ b/readline.c >> @@ -247,14 +247,14 @@ static void readline_hist_add(ReadLineState *rs, const >> char *cmdline) >> } >> if (idx == READLINE_MAX_CMDS) { >> /* Need to get one free slot */ >> - free(rs->history[0]); >> + g_free(rs->history[0]); >> memmove(rs->history, &rs->history[1], >> (READLINE_MAX_CMDS - 1) * sizeof(char *)); >> rs->history[READLINE_MAX_CMDS - 1] = NULL; >> idx = READLINE_MAX_CMDS - 1; >> } >> if (new_entry == NULL) >> - new_entry = strdup(cmdline); >> + new_entry = g_strdup(cmdline); > > As long as you are converting tabs to spaces, you should also be adding {}.
I style-fix exactly the lines I touch anyway. For what it's worth, checkpatch is happy.