Skip to content

Commit

Permalink
tidy: Fix simple warnings of 'value stored is never read' in freesurr…
Browse files Browse the repository at this point in the history
…ound.

The clang-tidy "dead stores" check pointed out several places where a
variables was set and never used.  It appears that his code used to be
part of a loop where the setting the variable would have made sense,
but now is only a single pass and the new value is never references.
Clean up these warnings by removing the assignments.
  • Loading branch information
linuxdude42 committed Mar 26, 2019
1 parent 8652e94 commit bdfc492
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions mythtv/libs/libmythfreesurround/freesurround.cpp
Expand Up @@ -355,7 +355,6 @@ uint FreeSurround::receiveFrames(void *buffer, uint maxFrames)
*output++ = *rs++;
}
oc -= maxFrames;
outindex += maxFrames;
}
else // channels == 6
{
Expand All @@ -378,7 +377,6 @@ uint FreeSurround::receiveFrames(void *buffer, uint maxFrames)
*output++ = *rs++;
}
oc -= maxFrames;
outindex += maxFrames;
}
else
{
Expand All @@ -398,7 +396,6 @@ uint FreeSurround::receiveFrames(void *buffer, uint maxFrames)
*output++ = *rs++;
}
oc -= maxFrames;
outindex += maxFrames;
}
}
out_count = oc;
Expand Down

0 comments on commit bdfc492

Please sign in to comment.