From patchwork Tue Apr 25 17:04:49 2006
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
X-Patchwork-Submitter: Klaus Schmidinger <Klaus.Schmidinger@cadsoft.de>
X-Patchwork-Id: 12279
Received: from raven.cadsoft.de ([217.7.101.211])
	by www.linuxtv.org with esmtp (Exim 4.50) id 1FYQy6-0003k8-1m
	for vdr@linuxtv.org; Tue, 25 Apr 2006 19:04:54 +0200
Received: from [192.168.100.10] (hawk.cadsoft.de [192.168.100.10])
	by raven.cadsoft.de (8.13.3/8.13.3) with ESMTP id k3PH4qmU030252
	for <vdr@linuxtv.org>; Tue, 25 Apr 2006 19:04:52 +0200
Message-ID: <444E56B1.7060708@cadsoft.de>
Date: Tue, 25 Apr 2006 19:04:49 +0200
From: Klaus Schmidinger <Klaus.Schmidinger@cadsoft.de>
Organization: CadSoft Computer GmbH
User-Agent: Thunderbird 1.5.0.2 (X11/20060420)
MIME-Version: 1.0
To: vdr@linuxtv.org
X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-2.0
	(raven.cadsoft.de [192.168.1.1]);
	Tue, 25 Apr 2006 19:04:52 +0200 (CEST)
Subject: [vdr] VDR 1.3.48: bug in cMenuEditIntItem
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: Tue, 25 Apr 2006 17:04:54 -0000
Status: O
X-Status: 
X-Keywords: 
X-UID: 9067

A small bug in cMenuEditIntItem has been fixed, which prevented
the display from being updated correctly when pressing the '0'
key.

Please review the attached patch to make sure there are no
unexpected side effects from this last minute fix.

Still expecting version 1.4.0 to be out next sunday... ;-)

Klaus

--- menuitems.c	2006/04/23 11:39:48	1.43
+++ menuitems.c	2006/04/25 15:59:02
@@ -80,10 +80,10 @@
        case kNone: break;
        case k0 ... k9:
             if (fresh) {
-               *value = 0;
+               newValue = 0;
                fresh = false;
                }
-            newValue = *value * 10 + (Key - k0);
+            newValue = newValue * 10 + (Key - k0);
             break;
        case kLeft: // TODO might want to increase the delta if repeated quickly?
             newValue = *value - 1;