Unused (assigned but not read) option in diff_no_index

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

I noticed that "options" variable in diff-no-index.c:diff_no_index()
is modified when "-q" is passed but not used (not read) at all. So
either it can be removed or it should be used. "-q" is not described
in git-diff man page.

This code was introduced in 0569e9b8 "git diff": do not ignore index
without --no-index" - it was moved from builtin-diff.c (where it was
used) but the code was refactored so I can't say if it should still
be.

$ grep -E -2 '\boptions\b' diff-no-index.c
	int i;
	int no_index = 0;
	unsigned options = 0;

	/* Were we asked to do --no-index explicitly? */
--
			i++;
		else if (!strcmp(argv[i], "-q")) {
			options |= DIFF_SILENT_ON_REMOVED;
			i++;
		}

-- 
Piotr Krukowiecki
--
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


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]