Skip to content

Commit

Permalink
tidy: Fix unused variables in jobqueue.cpp.
Browse files Browse the repository at this point in the history
The clang-tidy "dead stores" checker pointed out that the
jobsAreRunning variable is never checked after being set to false.
That is because the code immediately breaks out of the loop that is
checking this variable.  Replace the break with a continue, as this
has the same effect but reads better than completely removing the
jobsAreRunning variable.
  • Loading branch information
linuxdude42 committed Mar 26, 2019
1 parent cd8a45a commit 8c2a764
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion mythtv/libs/libmythtv/jobqueue.cpp
Expand Up @@ -810,7 +810,7 @@ bool JobQueue::DeleteAllJobs(uint chanid, const QDateTime &recstartts)
if (query.size() == 0)
{
jobsAreRunning = false;
break;
continue;
}
if ((totalSlept % 5) == 0)
{
Expand Down

0 comments on commit 8c2a764

Please sign in to comment.