mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-02-19 22:16:15 +00:00
Code Issues Projects Releases Packages Wiki Activity

Core: remove variable name conflict

Browse source 
Having a parameter with the same name as a global variable is potentially
confusing.

Found via LGTM.com

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Dirk Hohndel 2019-03-17 13:58:11 -07:00
parent b457965f50
commit 69248141c5
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
core/uemis-downloader.c
View file

@ -505,16 +505,16 @@ static void uemis_increased_timeout(int *timeout)
usleep(*timeout); usleep(*timeout);
} }
static char *build_ans_path(const char *path, int filenr) static char *build_ans_path(const char *path, int filenumber)
{ {
char *intermediate, *ans_path, fl[13]; char *intermediate, *ans_path, fl[13];
/* Clamp filenr into the 0..9999 range. This is never necessary, /* Clamp filenumber into the 0..9999 range. This is never necessary,
* as filenr can never go above UEMIS_MAX_FILES, but gcc doesn't * as filenumber can never go above UEMIS_MAX_FILES, but gcc doesn't
* recognize that and produces very noisy warnings. */ * recognize that and produces very noisy warnings. */
filenr = filenr < 0 ? 0 : filenr % 10000; filenumber = filenumber < 0 ? 0 : filenumber % 10000;
snprintf(fl, 13, "ANS%d.TXT", filenr); snprintf(fl, 13, "ANS%d.TXT", filenumber);
intermediate = build_filename(path, "ANS"); intermediate = build_filename(path, "ANS");
ans_path = build_filename(intermediate, fl); ans_path = build_filename(intermediate, fl);
free(intermediate); free(intermediate);