[mythtv-commits] Ticket #4870: Incorrect Aspect Ratio

[MythTV]

#4870: Incorrect Aspect Ratio
----------------------------+-----------------------------------------------
 Reporter:  greg            |        Owner:  ijr 
     Type:  defect          |       Status:  new 
 Priority:  minor           |    Milestone:  0.22
Component:  Video Playback  |      Version:  head
 Severity:  medium          |   Resolution:      
  Mlocked:  0               |  
----------------------------+-----------------------------------------------
Changes (by stuartm):

  * version:  unknown => head
  * component:  mythtv => Video Playback


Old description:

> When first starting playback of the test.avi video referenced in #4567
> we incorrectly set the aspect ratio to 2.   Later(even just 1 second
> later) if we for example seek back to the start of the video it is
> corrected, as well after a few seconds of playback we eventually get the
> right aspect and correct accordingly.
>

> From what I can see when we first call AvFormatDecoder::InitVideoCodec()
> enc->sample_aspect_ratio has no value and we end up defaulting to just a
> single width/height for the aspect.
>
>  But after a few frames have been decoded or, the next time
> AvFormatDecoder::H264PreProcessPkt gets called the sample_aspect_ratio is
> set so we can then properly calculate the aspect.
>
> I've tested this with other players and they don't seem to have the same
> problem.

New description:

 When first starting playback of the test.avi video referenced in #4567
 we incorrectly set the aspect ratio to 2.   Later(even just 1 second
 later) if we for example seek back to the start of the video it is
 corrected, as well after a few seconds of playback we eventually get the
 right aspect and correct accordingly.


 From what I can see when we first call AvFormatDecoder::InitVideoCodec()
 enc->sample_aspect_ratio has no value and we end up defaulting to just a
 single width/height for the aspect.

  But after a few frames have been decoded or, the next time
 AvFormatDecoder::H264PreProcessPkt gets called the sample_aspect_ratio is
 set so we can then properly calculate the aspect.

 I've tested this with other players and they don't seem to have the same
 problem.

--

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/4870#comment:1>
MythTV <http://www.mythtv.org/>
MythTV