Opened 13 years ago

Closed 12 years ago

#11274 closed Patch - Bug Fix (fixed)

mythpreviewgen: Allow screengrab without seek table

Reported by: managementboy@… Owned by: stuartm
Priority: minor Milestone: 0.27
Component: MythTV - Mythpreviewgen Version: Unspecified
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Stuart applied a patch from Lawrence at ​https://github.com/MythTV/mythtv/commit/9f992779383240c601fefdc7bd5049c2703e9959 but it is missing the last few lines of the diff. The previews only get created if hasFullPositionMap is not a requirement. 

--- mythtv-0.27.0~master.20121204.c05c1e1.orig/mythtv/libs/libmythtv/mythplayer.cpp                                           
+++ mythtv-0.27.0~master.20121204.c05c1e1/mythtv/libs/libmythtv/mythplayer.cpp                                                
@@ -4329,12 +4329,8 @@ void MythPlayer::SeekForScreenGrab(uint6                                                               
         }                                                                                                                    
     }                                                                                                                        
                                                                                                                              
-    // Only do seek if we have position map                                                                                  
-    if (hasFullPositionMap)                                                                                                  
-    {                                                                                                                        
-        DiscardVideoFrame(videoOutput->GetLastDecodedFrame());                                                               
-        DoJumpToFrame(number, kInaccuracyNone);                                                                              
-    }                                                                                                                        
+    DiscardVideoFrame(videoOutput->GetLastDecodedFrame());                                                                   
+    DoJumpToFrame(number, kInaccuracyNone);                                                                                  
 }

Change History (5)

comment:1 by Jim Stichnoth, 13 years ago

Owner: set to stuartm
Status: new → assigned

comment:2 by stuartm, 12 years ago

Milestone: unknown → 0.27
Status: assigned → accepted

comment:3 by stuartm, 12 years ago

Type: Bug Report - General → Patch - Feature

comment:4 by stuartm, 12 years ago

Type: Patch - Feature → Patch - Bug Fix

comment:5 by Stuart Morgan <smorgan@…>, 12 years ago

Resolution: → fixed
Status: accepted → closed

In ecd5c6f12d32273790b009b47385cfc493d72b08/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available
Note: See TracTickets for help on using tickets.