diff options
author | Dirk Hohndel <dirk@hohndel.org> | 2012-01-05 08:16:08 -0800 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2012-01-05 08:55:18 -0800 |
commit | c544226334ec1bfaac1c8d649722bb17728606e1 (patch) | |
tree | 87a9990dfe6f76d77a27f4adbe4c0b500e943ef7 /main.c | |
parent | 3d75c73f3677d98682b31766334ab02f124c76b9 (diff) | |
download | subsurface-c544226334ec1bfaac1c8d649722bb17728606e1.tar.gz |
Avoiding some potentially confusing name space clashes
We have local variables or function arguments with the same names as
function static variables (or in one case, function arguments).
While all the current code was correct, it could potentially cause
confusion when chasing bugs or reviewing patches. This should make things
clearer.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'main.c')
-rw-r--r-- | main.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -101,7 +101,7 @@ static gboolean imported = FALSE; * This doesn't really report anything at all. We just sort the * dives, the GUI does the reporting */ -void report_dives(gboolean imported) +void report_dives(gboolean is_imported) { int i; int preexisting = dive_table.preexisting; @@ -135,7 +135,7 @@ void report_dives(gboolean imported) i--; } - if (imported) { + if (is_imported) { /* Was the previous dive table state numbered? */ if (last && last->number) try_to_renumber(last, preexisting); |