Thread (2 messages) flat view 2 messages, 2 authors, 2017-06-29

Re: [PATCH v8 6/6] convert: add "status=delayed" to filter process protocol

From: Junio C Hamano <hidden>
Date: 2017-06-29 18:34:58

You seem to have squashed an unrelated "table-driven" clean-up into
this step.  While I think the end result looks good, I would have
liked to see it as a separate step, either as a preparatory "now we
are going to add the third one, let's make it table-driven before
doing so" step, or a follow-up "now we have done everything we
wanted to do, let's make this clean-up at the end" step.

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help