bucoord: Remove dead initialisation
authorSimon Wilkinson <sxw@your-file-system.com>
Sun, 24 Feb 2013 10:48:14 +0000 (10:48 +0000)
committerDerrick Brashear <shadow@your-file-system.com>
Mon, 25 Feb 2013 17:12:47 +0000 (09:12 -0800)
offset is simply used as a copy of nextOffset for the GetText
function call. We don't need to initialise it to 0 at the start of
the loop (and arguably, as it is passed to the function by value,
we could just remove the copy entirely)

Caught by clang-analyzer

Change-Id: I137ea3ba1ff44d94b90287d9735db69d6ff8717d
Reviewed-on: http://gerrit.openafs.org/9249
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>

src/bucoord/ubik_db_if.c

index 33c9b4e..fae1286 100644 (file)
@@ -495,7 +495,6 @@ bcdb_GetTextFile(udbClientTextP ctPtr)
        ERROR(BUDB_INTERNALERROR);
     charList.charListT_len = bufferSize;
 
-    offset = 0;
     nextOffset = 0;
     ctPtr->textSize = 0;
     while (nextOffset != -1) {