Fix ?: statements.
Omitting the first branch in ?: is a GNU extension. Cute, but not supported by other compilers. Replaced mostly by explicit tests. Calls to getenv() simply are repeated on non-GNU compilers. Signed-off-by: Jason Riedy <ejr@cs.berkeley.edu>
This commit is contained in:
4
path.c
4
path.c
@ -41,10 +41,12 @@ char *mkpath(const char *fmt, ...)
|
||||
|
||||
char *git_path(const char *fmt, ...)
|
||||
{
|
||||
const char *git_dir = gitenv(GIT_DIR_ENVIRONMENT) ? : DEFAULT_GIT_DIR_ENVIRONMENT;
|
||||
const char *git_dir;
|
||||
va_list args;
|
||||
unsigned len;
|
||||
|
||||
git_dir = gitenv(GIT_DIR_ENVIRONMENT);
|
||||
if (!git_dir) git_dir = DEFAULT_GIT_DIR_ENVIRONMENT;
|
||||
len = strlen(git_dir);
|
||||
if (len > PATH_MAX-100)
|
||||
return bad_path;
|
||||
|
Reference in New Issue
Block a user