On Sun, 2014-03-09 at 18:42 -0700, Linus Torvalds wrote: > The argument order is a bit mixed up, which is partly why this happened. > But considering that this is the worst bug I've hit so far in the saving > code, I guess I shouldn't complain too much. > > Signed-off-by: Linus Torvalds <torva...@linux-foundation.org> > --- > save-git.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/save-git.c b/save-git.c > index 7b121a19b7e6..c47638b8e48e 100644 > --- a/save-git.c > +++ b/save-git.c > @@ -292,7 +292,7 @@ static void save_one_event(struct membuffer *b, struct > event *ev) > show_index(b, ev->type, "type=", ""); > show_index(b, ev->flags, "flags=", ""); > show_index(b, ev->value, "value=", ""); > - show_utf8(b, ev->name, " name=", ""); > + show_utf8(b, " name=", ev->name, ""); > put_string(b, "\n");
wouldn't it be better to have consistent argument order? /D _______________________________________________ subsurface mailing list subsurface@hohndel.org http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface