Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes: > It used to ignore the return value of the helper function; now, it > expects it to return 0, and stops iteration upon non-zero return > values; this value is then passed on as the return value of > for_each_reflog_ent(). Good idea. > Further, it makes no sense to force the parsing upon the helper > functions; for_each_reflog_ent() now calls the helper function with > old and new sha1, the email, the timestamp & timezone, and the message. Perhaps. I did it that way deliberately because all existing users did not have to pay overhead of parsing. Thanks. Will queue in 'pu' initially as I do not have much time to look at it tonight. - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html