From patchwork Sun Feb 19 17:59:00 2006
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
X-Patchwork-Submitter: Christoph Haubrich <christoph1.haubrich@arcor.de>
X-Patchwork-Id: 12209
Received: from mail-in-09.arcor-online.net ([151.189.21.49])
	by www.linuxtv.org with esmtp (Exim 4.50) id 1FAsq1-00084w-DK
	for vdr@linuxtv.org; Sun, 19 Feb 2006 18:59:13 +0100
Received: from mail-in-09-z2.arcor-online.net (mail-in-09-z2.arcor-online.net
	[151.189.8.21])
	by mail-in-09.arcor-online.net (Postfix) with ESMTP id EE19E22BF2F
	for <vdr@linuxtv.org>; Sun, 19 Feb 2006 18:59:11 +0100 (CET)
Received: from mail-in-05.arcor-online.net (mail-in-05.arcor-online.net
	[151.189.21.45])
	by mail-in-09-z2.arcor-online.net (Postfix) with ESMTP id E18F28A26A
	for <vdr@linuxtv.org>; Sun, 19 Feb 2006 18:59:11 +0100 (CET)
Received: from [192.168.1.2] (p54AE87EB.dip0.t-ipconnect.de [84.174.135.235])
	(Authenticated sender: christoph1.haubrich@arcor.de)
	by mail-in-05.arcor-online.net (Postfix) with ESMTP id 7CCB011E2FD
	for <vdr@linuxtv.org>; Sun, 19 Feb 2006 18:59:11 +0100 (CET)
Message-ID: <43F8B1E4.9000107@arcor.de>
Date: Sun, 19 Feb 2006 18:59:00 +0100
From: Christoph Haubrich <christoph1.haubrich@arcor.de>
User-Agent: Thunderbird 1.5 (X11/20051201)
MIME-Version: 1.0
To: VDR Mailing List <vdr@linuxtv.org>
Subject: [vdr] [PATCH] replayjump-ok
X-BeenThere: vdr@linuxtv.org
X-Mailman-Version: 2.1.5
Precedence: list
Reply-To: VDR Mailing List <vdr@linuxtv.org>
List-Id: VDR Mailing List <vdr.linuxtv.org>
List-Unsubscribe: <http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr>,
	<mailto:vdr-request@linuxtv.org?subject=unsubscribe>
List-Archive: <http://www.linuxtv.org/pipermail/vdr>
List-Post: <mailto:vdr@linuxtv.org>
List-Help: <mailto:vdr-request@linuxtv.org?subject=help>
List-Subscribe: <http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr>,
	<mailto:vdr-request@linuxtv.org?subject=subscribe>
X-List-Received-Date: Sun, 19 Feb 2006 17:59:13 -0000
Status: O
X-Status: 
X-Keywords: 
X-UID: 8001

Hello list,

I was always stumbling over the behaviour of jumping in recordings (red 
button): The OK-button does not confirm the jump but aborts it which I 
think is rather unlogical.
The appended patch gives the OK-Button the same behaviour as the "down" 
and "pause" buttons, which means it jumps to the requested position and 
pauses the replay.

If no one complains I would like Klaus to integrate this patch into vdr.

Christoph

diff -Nur vdr-1.3.32-pre/menu.c vdr-1.3.32/menu.c
--- vdr-1.3.32-pre/menu.c	2005-09-18 13:53:30.000000000 +0200
+++ vdr-1.3.32/menu.c	2005-09-18 14:11:59.000000000 +0200
@@ -3740,8 +3740,9 @@
     case kUp:
     case kPause:
     case kDown:
+    case kOk:
          Seconds = min(Total - STAY_SECONDS_OFF_END, Seconds);
-         Goto(Seconds * FRAMESPERSEC, Key == kDown || Key == kPause);
+         Goto(Seconds * FRAMESPERSEC, Key == kDown || Key == kPause || Key == kOk);
          timeSearchActive = false;
          break;
     default: