Skip to content

Commit c55c3f2

Browse files
newrengitster
authored andcommitted
merge-ort: loosen commented requirements
The comment above type_short_descriptions claimed that the order had to match what was found in the conflict_info_and_types enum. Since type_short_descriptions uses designated initializers, the order should not actually matter; I am guessing that positional initializers may have been under consideration when that comment was added, but the comment was not updated when designated initializers were chosen. Signed-off-by: Elijah Newren <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 5fadf1f commit c55c3f2

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

merge-ort.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -553,7 +553,7 @@ enum conflict_and_info_types {
553553
* Short description of conflict type, relied upon by external tools.
554554
*
555555
* We can add more entries, but DO NOT change any of these strings. Also,
556-
* Order MUST match conflict_info_and_types.
556+
* please ensure the order matches what is used in conflict_info_and_types.
557557
*/
558558
static const char *type_short_descriptions[] = {
559559
/*** "Simple" conflicts and informational messages ***/

0 commit comments

Comments
 (0)