Junio C Hamano <gitster@xxxxxxxxx> writes: > Perhaps > > if (value && !strcmp(value, "batch")) > fsync_object_files = FSYNC_OBJECT_FILES_BATCH; > else if (git_config_bool(var, value)) > fsync_object_files = FSYNC_OBJECT_FILES_ON; > else > fsync_object_files = FSYNC_OBJECT_FILES_OFF; By the way, in case it wasn't clear, I do mean strcmp and not strcasecmp. Making these things that are meant to be machine readable tokens to be spelled in different ways in the name of "friendliness" is a disease. Thanks.