[mythtv] [mythtv-commits] mythtv branch devel/027candidates updated by jyavenard. v0.27-322-gfc6e69a

[John Pilkington]

On 19/05/14 00:13, Git Repo Owner wrote:
> The branch, devel/027candidates has been updated on the
> mythtv repository by gitolite user jyavenard.
>         via  fc6e69a3e8aaeb8c956a7e01ec170f1b7c073f36 (commit)
>         via  ab576e79b7a1e04673349191482fa700e5c476b0 (commit)
>        from  db0cdc8608a72b04b20721c493a94d10d3b002de (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 fc6e69a3e8aaeb8c956a7e01ec170f1b7c073f36
> Author:    Jean-Yves Avenard <jyavenard at mythtv.org> at Mon, 19 May 2014 02:31:20 +1000
> Committer: Jean-Yves Avenard <jyavenard at mythtv.org> at Mon, 19 May 2014 09:11:58 +1000
> URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=fc6e69a3e8aaeb8c956a7e01ec170f1b7c073f36
>
> Improve FastForward and Rewind
> This improves seeks in in-progress recordings, retrieving up to date duration from the backend only when absolutely required
> When fast forwarding, we jump to -3s before the end at the most (-1s otherwise)
>
> Fixes #12146
>
> (cherry picked from commit 244ce4feecd1df80c915023540a73684140d037a)
>
>
>
> commit ab576e79b7a1e04673349191482fa700e5c476b0
> Author:    Jean-Yves Avenard <jyavenard at mythtv.org> at Mon, 19 May 2014 01:37:46 +1000
> Committer: Jean-Yves Avenard <jyavenard at mythtv.org> at Mon, 19 May 2014 09:08:17 +1000
> URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=ab576e79b7a1e04673349191482fa700e5c476b0
>
> Reset frames/duration maps when changing file, or when initialising the position map.
> frame/position maps were never reset, causing the PositionMap to rarely been updated and the totalFrames and program duration to never be updated after the file had been opened.
> This was responsible for wrong seeks in in-progress recordings and liveTV
>
> (cherry picked from commit 632cf60862dd106f89db4121b7c592e37694cd58)
>
>
>
> -----------------------------------------------------------------------
>
> Summary of changes:
>   mythtv/libs/libmythtv/decoderbase.cpp |    4 ++
>   mythtv/libs/libmythtv/mythplayer.cpp  |   67 ++++++++++++++++++++++++++-------
>   2 files changed, 57 insertions(+), 14 deletions(-)
>
I just built 0.27.1-11-ga1b5067 on my SL6 laptop.  I don't have a build 
log but a warning caught my eye about a comparison between signed and 
unsigned integers at line 4822 in mythplayer.cpp

I don't know how significant that is.  It's here:

long long framesWritten = player_ctx->recorder->GetFramesWritten();

             if (totalFrames < framesWritten)

John P