[MPlayer-dev-eng] Found a bug in some slave mode code

[Jeremy Lawler]

I wrote some code that utilizes slave mode, and it has come to my attention
that "get_time_pos" has at least 1 bug.
The "get_time_pos" command  returns the value of sh_video->pts.
 Unfortunately, during DVD playback this is the position within the current
VOB file, not the total DVD stream.

I ended up using the demux->stream_pts to grab the info because the
demuxer_get_current_time function returns ints, not floats.  I assumed it
would be preferred to keep the current precision instead of using
get_current_time.

I joined the mailing list to post this and attempted to make sure my patch
is minimal/formatted correctly, so if there is anything wrong with the
patch/logic, apologies.


Jeremy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer_fix_get_time_pos.diff
Type: text/x-patch
Size: 601 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20110904/621f55ca/attachment.bin>