 338abb0f04
			
		
	
	338abb0f04
	
	
	
		
			
			Change various cmd_* functions that claim to return an "int" to use
"return" instead of exit() to indicate an exit code. These were not
marked with NORETURN, and by directly exit()-ing we'll skip the
cleanup git.c would otherwise do (e.g. closing fd's, erroring if we
can't). See run_builtin() in git.c.
In the case of shell.c and sh-i18n--envsubst.c this was the result of
an incomplete migration to using a cmd_main() in 3f2e2297b9 (add an
extra level of indirection to main(), 2016-07-01).
This was spotted by SunCC 12.5 on Solaris 10 (gcc210 on the gccfarm).
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
		
	
		
			
				
	
	
		
			34 lines
		
	
	
		
			881 B
		
	
	
	
		
			C
		
	
	
	
	
	
			
		
		
	
	
			34 lines
		
	
	
		
			881 B
		
	
	
	
		
			C
		
	
	
	
	
	
| /*
 | |
|  * Implementation of git-merge-ours.sh as builtin
 | |
|  *
 | |
|  * Copyright (c) 2007 Thomas Harning Jr
 | |
|  * Original:
 | |
|  * Original Copyright (c) 2005 Junio C Hamano
 | |
|  *
 | |
|  * Pretend we resolved the heads, but declare our tree trumps everybody else.
 | |
|  */
 | |
| #define USE_THE_INDEX_COMPATIBILITY_MACROS
 | |
| #include "git-compat-util.h"
 | |
| #include "builtin.h"
 | |
| #include "diff.h"
 | |
| 
 | |
| static const char builtin_merge_ours_usage[] =
 | |
| 	"git merge-ours <base>... -- HEAD <remote>...";
 | |
| 
 | |
| int cmd_merge_ours(int argc, const char **argv, const char *prefix)
 | |
| {
 | |
| 	if (argc == 2 && !strcmp(argv[1], "-h"))
 | |
| 		usage(builtin_merge_ours_usage);
 | |
| 
 | |
| 	/*
 | |
| 	 * The contents of the current index becomes the tree we
 | |
| 	 * commit.  The index must match HEAD, or this merge cannot go
 | |
| 	 * through.
 | |
| 	 */
 | |
| 	if (read_cache() < 0)
 | |
| 		die_errno("read_cache failed");
 | |
| 	if (index_differs_from(the_repository, "HEAD", NULL, 0))
 | |
| 		return 2;
 | |
| 	return 0;
 | |
| }
 |