Skip to content

Commit 44da7ec

Browse files
committed
main,refactor: don't use else if if-block does return always
Signed-off-by: Masatake YAMATO <[email protected]>
1 parent 14791ce commit 44da7ec

File tree

1 file changed

+16
-19
lines changed

1 file changed

+16
-19
lines changed

main/options.c

Lines changed: 16 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1777,17 +1777,16 @@ static char* extractMapFromParameter (const langType language,
17771777
*tail = parameter + strlen (parameter);
17781778
return result;
17791779
}
1780-
else
1781-
{
1782-
tmp = *p;
1783-
*p = '\0';
1784-
result = eStrdup (parameter);
1785-
*p = tmp;
1786-
*tail = p;
1787-
return result;
1788-
}
1780+
1781+
tmp = *p;
1782+
*p = '\0';
1783+
result = eStrdup (parameter);
1784+
*p = tmp;
1785+
*tail = p;
1786+
return result;
17891787
}
1790-
else if (first == PATTERN_START) /* pattern map */
1788+
1789+
if (first == PATTERN_START) /* pattern map */
17911790
{
17921791
*mapType = LMAP_PATTERN;
17931792

@@ -1800,15 +1799,13 @@ static char* extractMapFromParameter (const langType language,
18001799
if (*p == '\0')
18011800
error (FATAL, "Unterminated file name pattern for %s language",
18021801
getLanguageName (language));
1803-
else
1804-
{
1805-
tmp = *p;
1806-
*p = '\0';
1807-
result = eStrdup (parameter);
1808-
*p = tmp;
1809-
*tail = p + 1;
1810-
return result;
1811-
}
1802+
1803+
tmp = *p;
1804+
*p = '\0';
1805+
result = eStrdup (parameter);
1806+
*p = tmp;
1807+
*tail = p + 1;
1808+
return result;
18121809
}
18131810

18141811
return NULL;

0 commit comments

Comments
 (0)