[mythtv-commits] Ticket #10333: Commit 8038cb11da4e4f803e88f4476d9e49348bb3ca19 introduces various closed caption bugs

[MythTV]

#10333: Commit 8038cb11da4e4f803e88f4476d9e49348bb3ca19 introduces various closed
caption bugs
----------------------------------------------+----------------------------
 Reporter:  Clayton Smith <argilo@…>          |          Owner:  stichnot
     Type:  Bug Report - General              |         Status:  accepted
 Priority:  minor                             |      Milestone:  0.25
Component:  MythTV - Captions                 |        Version:  Master
 Severity:  medium                            |  Head
 Keywords:  cea-708                           |     Resolution:
                                              |  Ticket locked:  0
----------------------------------------------+----------------------------

Comment (by stichnot):

 We appear to be within the spec for text height with the standard pen
 size.  In both the old code and the new code, the font height is set to
 1/20 of the safe area height.  In the new code, additional spacing is
 added between lines such that the distance from the top of one line to the
 top of the next line is 1/17 of the safe area height.  (Note that 17 is
 chosen to match the number of rows in the non-teletext CEA-608 spec.)
 This is all assuming OSDCC708TextZoom is set to 100.

 On top of this, there is a 4% slop added to deal with what looks like a Qt
 text size calculation bug, plus possible cumulative truncation/rounding
 error, but I would be very surprised if this reduced 17 lines down to less
 than the spec's minimum 15.

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/10333#comment:5>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center