Re: [PATCH v3 09/20] commit-graph: verify corrupt OID fanout and lookup
From: Derrick Stolee <hidden>
Date: 2018-06-04 11:32:56
From: Derrick Stolee <hidden>
Date: 2018-06-04 11:32:56
On 6/2/2018 12:38 AM, Duy Nguyen wrote:
On Thu, May 24, 2018 at 6:25 PM, Derrick Stolee [off-list ref] wrote:quoted
+ if (i && oidcmp(&prev_oid, &cur_oid) >= 0) + graph_report("commit-graph has incorrect OID order: %s then %s", + oid_to_hex(&prev_oid), + oid_to_hex(&cur_oid));Should these strings be marked for translation with _()?
I've been asking myself "Is this message helpful to anyone other than a Git developer?" and for this series the only one that is helpful to an end-user is the message about the final hash. If the hash is correct, but these other messages appear, then there is a bug in the code that wrote the file. Otherwise, file corruption is more likely and the correct course of action is to delete and rebuild. Thanks for being diligent in checking. Thanks, -Stolee