Avidemux Forum

Avidemux => Main version 2.6 => Topic started by: zylantha on July 22, 2012, 06:38:05 AM

Title: Keyboard-controlled fast seeking broken
Post by: zylantha on July 22, 2012, 06:38:05 AM
The code for fast keyboard navigation through the media file is not currently working, i.e. in QT4 Q_gui2.cpp there is code for:
                                        case Qt::Key_Left:
                                                if (keyEvent->modifiers() == Qt::ShiftModifier)
                                                        HandleAction(ACT_Back25Frames);
                                                else if (keyEvent->modifiers() == Qt::ControlModifier)
                                                        HandleAction(ACT_Back50Frames);
                                                else
                                                        HandleAction(ACT_PreviousFrame);

                                                return true;

and similarly for forward, however these do not currently work.  Is there an easy answer 'why' and how to fix it?  I am trying to get it working again as this functionality is essential for my workflow and migration to 2.6.

From what I can tell, it seems as though this code is never executed, and that keyboard events are now handled somewhere else that I am yet to find.
Title: Re: Keyboard-controlled fast seeking broken
Post by: mean on July 22, 2012, 06:41:54 AM
2.5 was thinking in  # of frames
2.6 is thinking in time

So that part has to be adapted and 50frame forward replaced by 0.5 sec etc..
Title: Re: Keyboard-controlled fast seeking broken
Post by: zylantha on July 22, 2012, 08:32:32 AM
Thanks mean

After an afternoon of tinkering, I've made a bunch of changes that have this seeking functionality adapted to equivalent time-based notion, and it looks to be working properly now.

Modified files are attached.

Nb: I have also removed some warning messages that shouldn't be there (PrevFrame failures - which are typically because you're at the beginning of the file) and hacked the 'End' seek because it was also failing ... there is undoubtedly a better way of doing this..
Title: Re: Keyboard-controlled fast seeking broken
Post by: mean on July 22, 2012, 10:08:47 AM
Commited, thanks a lot
Title: Re: Keyboard-controlled fast seeking broken
Post by: zylantha on July 22, 2012, 10:25:55 AM
Thanks mean

Further update with additional keyboard shortcuts for jumping to beginning and end markers ...
Title: Re: Keyboard-controlled fast seeking broken
Post by: Jan Gruuthuse on July 23, 2012, 05:42:08 AM
Quote from: zylanthaI've made a bunch of changes that have this seeking functionality adapted to equivalent time-based notion, and it looks to be working properly now.
Quote from: meanCommited, thanks a lot

(https://avidemux.org/smf/Themes/default/images/post/thumbup.gif)

goto end (https://avidemux.org/smif/proxy.php?request=http%3A%2F%2Fwww.avidemux.org%2FadmWiki%2Flib%2Fexe%2Ffetch.php%3Fmedia%3Dusing%3Alast_frame-qt.png&hash=2d20e58f7a8042493c4457e42b66cd381c642904) and beginning (https://avidemux.org/smif/proxy.php?request=http%3A%2F%2Fwww.avidemux.org%2FadmWiki%2Flib%2Fexe%2Ffetch.php%3Fmedia%3Dusing%3Afirst_frame-qt.png&hash=259e563672c5e389db15a6a886df5e39b33b5fb7) works now without the errors, trying to pass beyond these, in mpeg-ts streams.

thank you both
Title: Re: Keyboard-controlled fast seeking broken
Post by: zylantha on July 23, 2012, 11:01:16 AM
The fix for 'go to end' was a bit of a hack.  Hell it was a lot of a hack ... it doesn't actually go quite all the way to the end.  I just subtracted enough time off the video duration to stop the error message appearing - once you do go to the end, you can still keep framing forward a little - not much, in my case it seems to only be 10 frames or so. 

There is to bound to be a better way to do this, but at least it works now.
Title: Re: Keyboard-controlled fast seeking broken
Post by: Jan Gruuthuse on August 13, 2012, 10:01:58 AM
Is it some how possible this hack is no longer working/enabled/... in the 2.6?