[mythtv-commits] mythtv branch fixes/0.25 updated by stuartm. v0.25-98-gfb66b95

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


The branch, fixes/0.25 has been updated on the
mythtv repository by gitolite user stuartm.
       via  fb66b95ff7b87025c680dd9b4432984b2e92d9de (commit)
       via  da5ea5fbda31b951bf7cf91f0f0c85d05e608ccd (commit)
       via  08cb1559116da1823381f6c6aeda28537bdea4e6 (commit)
      from  b90b71b2695305136a3b0b24384df0a58019ec4d (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 fb66b95ff7b87025c680dd9b4432984b2e92d9de
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:45:05 +0100
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=fb66b95ff7b87025c680dd9b4432984b2e92d9de

Fix assignment where comparison was intended in FreeSurround::getLatency(). Caught by Coverity (cherry picked from commit c85ed06d244eaf02b29771d5122166e3d8ef19be)



commit da5ea5fbda31b951bf7cf91f0f0c85d05e608ccd
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:10:13 +0100
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=da5ea5fbda31b951bf7cf91f0f0c85d05e608ccd

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. (cherry picked from commit e84a3aaa2e1f4f744d08e31809ec1852429b7af5)



commit 08cb1559116da1823381f6c6aeda28537bdea4e6
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 22:26:15 +0100
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=08cb1559116da1823381f6c6aeda28537bdea4e6

Add missing break statement in PlaybackBox::ShowAvailabilityPopup(). Caught by Coverity. (cherry picked from commit 9d843edc58edd921b17d4b3b374056451af72536)



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

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-commits mailing list