summaryrefslogtreecommitdiffstats
path: root/main.c
diff options
context:
space:
mode:
authorGravatar Dirk Hohndel <dirk@hohndel.org>2012-01-05 08:16:08 -0800
committerGravatar Dirk Hohndel <dirk@hohndel.org>2012-01-05 08:55:18 -0800
commitc544226334ec1bfaac1c8d649722bb17728606e1 (patch)
tree87a9990dfe6f76d77a27f4adbe4c0b500e943ef7 /main.c
parent3d75c73f3677d98682b31766334ab02f124c76b9 (diff)
downloadsubsurface-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.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/main.c b/main.c
index d77fcd2d9..67e19b02b 100644
--- a/main.c
+++ b/main.c
@@ -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);