From 2024041b6c5134a925a33c10eff24a47ecb541a6 Mon Sep 17 00:00:00 2001 From: Richard Yao Date: Wed, 23 Apr 2014 00:04:41 -0400 Subject: [PATCH] Remove superfluous statement Clang's static analyzer reported that the value assigned to pcksum is never used. That is because we initialize both zc and pcksum to {{ 0 }} and then do `pcksum = zc;`. That is fairly pointless. However, it has the effect of generating a false positive in Clang's static analyzer. Since noise from false positives can obscure real issues, we fix it anyway. Signed-off-by: Richard Yao Signed-off-by: Ned Bass Signed-off-by: Brian Behlendorf Issue #2330 --- cmd/zstreamdump/zstreamdump.c | 1 - 1 file changed, 1 deletion(-) diff --git a/cmd/zstreamdump/zstreamdump.c b/cmd/zstreamdump/zstreamdump.c index c5bcdabd3c..a4c451d530 100644 --- a/cmd/zstreamdump/zstreamdump.c +++ b/cmd/zstreamdump/zstreamdump.c @@ -127,7 +127,6 @@ main(int argc, char *argv[]) } send_stream = stdin; - pcksum = zc; while (ssread(drr, sizeof (dmu_replay_record_t), &zc)) { if (first) {