Junio C Hamano <[email protected]> writes:
> I ran out of time for the day and haven't looked at individual
> patches yet. I may find other issues, but from the interdiff, that
> was the only thing that I found even worse than the previous round,
> and everything else was either the same or has improved.
... I forgot to say a few things:
* In the meantime, I think this is overall better than what we had
for the past few weeks; I'll try to replace the copy in my tree
with this round, even though this may not be the final one.
* There were a few style issues [*1*] pointed out by the
checkpatch.pl script borrowed from the kernel land.
* And finally, you wrote "This marks the count down to '3': two
more patch series after this (really tiny ones) and we have a
faster rebase -i." This is seriously an exciting news. Thanks.
I probably am still forgetting to say a few more things, but I'd
send this message out in this shape for now. Thanks.
[Footnote]
*1*
Subject: [PATCH v3 10/38] sequencer (rebase -i): add support for the 'fixup'
and 'squash' commands
ERROR: do not use assignment in if condition
#118: FILE: sequencer.c:725:
+ if (buf.buf[0] != comment_line_char ||
ERROR: do not use assignment in if condition
#142: FILE: sequencer.c:749:
+ if (!(head_commit = lookup_commit_reference(head)))
ERROR: do not use assignment in if condition
#144: FILE: sequencer.c:751:
+ if (!(head_message = get_commit_buffer(head_commit, NULL)))
ERROR: do not use assignment in if condition
#167: FILE: sequencer.c:774:
+ if (!(message = get_commit_buffer(commit, NULL)))
total: 4 errors, 1 warnings, 320 lines checked
------------------------------------------------
Subject: [PATCH v3 12/38] sequencer (rebase -i): write an author-script file
ERROR: do not use assignment in if condition
#35: FILE: sequencer.c:500:
+ else if ((eol = strchr(message, '\n')))
total: 1 errors, 0 warnings, 62 lines checked
------------------------------------------------
Subject: [PATCH v3 23/38] sequencer (rebase -i): copy commit notes at end
ERROR: open brace '{' following function declarations go on the next line
#36: FILE: sequencer.c:862:
+static void flush_rewritten_pending(void) {
ERROR: do not use assignment in if condition
#41: FILE: sequencer.c:867:
+ if (strbuf_read_file(&buf, rebase_path_rewritten_pending(), 82) > 0 &&
total: 2 errors, 0 warnings, 112 lines checked
