[mythtv-commits] Ticket #10402: Alternative implementation of CC608zoom; Implement DVD subtitle zoom

MythTV noreply at mythtv.org
Mon May 21 17:30:12 UTC 2012


#10402: Alternative implementation of CC608zoom; Implement DVD subtitle zoom
-------------------------------+-----------------------------
 Reporter:  ggervasio@…        |          Owner:  stichnot
     Type:  Patch - Feature    |         Status:  accepted
 Priority:  minor              |      Milestone:  0.26
Component:  MythTV - Captions  |        Version:  Master Head
 Severity:  medium             |     Resolution:
 Keywords:                     |  Ticket locked:  0
-------------------------------+-----------------------------

Comment (by stichnot):

 Gregorio,

 I have a couple of questions about the AV subtitle patch.  First, this
 portion of the patch:

 {{{
 @@ -269,7 +272,8 @@ void SubtitleScreen::DisplayAVSubtitles(void)
                  int right  = rect->x + rect->w;
                  int bottom = rect->y + rect->h;
                  if (subs->fixPosition || (currentFrame->height < bottom)
 ||
 -                   (currentFrame->width  < right))
 +                   (currentFrame->width  < right) ||
 +                   !display.width() || !display.height())
                  {
                      int sd_height = 576;
                      if ((m_player->GetFrameRate() > 26.0f) && bottom <=
 480)

 }}}
 doesn't seem to have anything to do with zooming, right?  Do you have a
 sample that displays incorrectly (irrespective of zooming) when width or
 height are 0?

 Second, comments in the patch indicate that if a subtitle spans the center
 line, it gets split into the top half and the bottom half, and each half
 is drawn separately.  It seems that that would work reasonably for a zoom
 greater than 100%, but that a zoom less than 100% would lead to a possibly
 large gap between the two halves.  Is this right?  Do you have a sample
 with a subtitle spanning the center line?

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


More information about the mythtv-commits mailing list