Re: [PATCH v3 2/3] add-patch: Allow interfile navigation when selecting hunks

Abraham Samuel Adekunle [off-list ref] writes:

-		} else if (s->answer.buf[0] == 'K') {
+		} else if (s->s.no_auto_advance && s->answer.buf[0] == '>') {
+			if (permitted & ALLOW_GOTO_NEXT_FILE) {
+				ret = NEXT_FILE;
+...
+				continue;
+			}
+		}
+		else if (s->answer.buf[0] == 'K') {

This funny-looking diff is a sign that the coding guideline was
followed in the preimage but not in the postimage, by splitting
the "} else if (condition) {" into two lines for 'K'.

On Fri, Feb 6, 2026 at 7:35 PM Junio C Hamano [off-list ref] wrote:

Abraham Samuel Adekunle [off-list ref] writes:

quoted

-             } else if (s->answer.buf[0] == 'K') {
+             } else if (s->s.no_auto_advance && s->answer.buf[0] == '>') {
+                     if (permitted & ALLOW_GOTO_NEXT_FILE) {
+                             ret = NEXT_FILE;
+...
+                             continue;
+                     }
+             }
+             else if (s->answer.buf[0] == 'K') {

This funny-looking diff is a sign that the coding guideline was
followed in the preimage but not in the postimage, by splitting
the "} else if (condition) {" into two lines for 'K'.

Sorry, I will fix it