Junio C Hamano <gits...@pobox.com> writes:

> Jeff King <p...@peff.net> writes:
>> However, the shared bits are simple enough that maybe that is not a
>> concern. An interesting test would be to add a 5/4 porting Erik's win32
>> credential helper, since that is the platform least like our other ones.
> Very true.
>> So I am OK with this series, but I am also OK with leaving it at patch
>> 1, and just keeping the implementations separate.
> Amen.

Just to make sure we do not leave loose ends, could somebody try to
see if the new "generic helper" infrastructure is useful to shrink
Erik's win32 credential helper implementation?

If we see much code reduction and improved clarity, this refactoring
may worth keeping.  Otherwise it may be sufficient to drop the later
ones in the series.  Without knowing which, it is hard to decide.

To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to