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>

1 files changed, 3 insertions, 3 deletions

diff --git a/info.c b/info.c
index e53af015e..6a4a296b0 100644
--- a/info.c
+++ b/info.c
@@ -258,13 +258,13 @@ void add_location(const char *string)
 
 static int get_rating(const char *string)
 {
-	int rating = 0;
+	int rating_val = 0;
 	int i;
 
 	for (i = 0; i <= 5; i++)
 		if (!strcmp(star_strings[i],string))
-			rating = i;
-	return rating;
+			rating_val = i;
+	return rating_val;
 }
 
 struct dive_info {