[mythtv-firehose] mythtv branch master updated by stuartm. v0.26-pre-317-gc85ed06

Git Repo Owner noreply at mythtv.org
Fri May 18 22:54:06 UTC 2012


The branch, master has been updated on the
mythtv repository by gitolite user stuartm.
       via  c85ed06d244eaf02b29771d5122166e3d8ef19be (commit)
       via  02f546356385dfbc6c72191080b3f3fa6d8b3311 (commit)
       via  fae8d9f10d5190eb466f4d582efaf8c82f9b0dca (commit)
      from  acc4cd4bc2cf4f320b56d8ecdfaf737af047bbfe (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit c85ed06d244eaf02b29771d5122166e3d8ef19be
Author:    Stuart Morgan <smorgan at mythtv.org> at Fri, 18 May 2012 23:44:45 +0100
Committer: Stuart Morgan <smorgan at mythtv.org> at Fri, 18 May 2012 23:44:45 +0100
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=c85ed06d244eaf02b29771d5122166e3d8ef19be

Fix assignment where comparison was intended in FreeSurround::getLatency(). Caught by Coverity



commit 02f546356385dfbc6c72191080b3f3fa6d8b3311
Author:    Stuart Morgan <smorgan at mythtv.org> at Fri, 18 May 2012 23:09:27 +0100
Committer: Stuart Morgan <smorgan at mythtv.org> at Fri, 18 May 2012 23:14:57 +0100
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=02f546356385dfbc6c72191080b3f3fa6d8b3311

Fix assumption that ints would be initialised to zero in JobQueueEntry, this isn't the case with C++. It appears that this would have caused any job that didn't include a chanid to abort even though the code indicates that it's perfectly valid to have a job which doesn't operate on a recording and wouldn't require the chanid. I don't know enough about the intended behaviour to say whether this will fix a user-visible bug ... Discovered by Coverity.



commit fae8d9f10d5190eb466f4d582efaf8c82f9b0dca
Author:    Stuart Morgan <smorgan at mythtv.org> at Fri, 18 May 2012 22:24:26 +0100
Committer: Stuart Morgan <smorgan at mythtv.org> at Fri, 18 May 2012 23:14:56 +0100
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=fae8d9f10d5190eb466f4d582efaf8c82f9b0dca

Add missing break statement in PlaybackBox::ShowAvailabilityPopup(). Caught by Coverity.



-----------------------------------------------------------------------

Summary of changes:
 mythtv/libs/libmythfreesurround/freesurround.cpp |    2 +-
 mythtv/libs/libmythtv/jobqueue.cpp               |    1 +
 mythtv/programs/mythfrontend/playbackbox.cpp     |    1 +
 3 files changed, 3 insertions(+), 1 deletions(-)

-- 



More information about the mythtv-firehose mailing list