Bug in text2skin [PATCH]

Message ID 435C6B51.6050902@syphir.sytes.net
State New
Headers

Commit Message

C.Y.M Oct. 24, 2005, 5:04 a.m. UTC
  C.Y.M wrote:
> Anssi Hannula wrote:
> 
>>C.Y.M wrote:
>>
>>
>>>Yes, I still get the problem with a "plain" copy of Enigma-skin.  For
>>>me, the
>>>problem started to occur when I upgraded to text2skin 1.0.
>>
>>
>>I now see the problem too, but it doesn't occur always:
>>If I want to go channel 1472, I first press 1, then 4. At this point, if
>>channel 14 exists, there's no problem, but if channel 14 doesn't exist,
>>the number doesn't show in the screen. The number returns when I press
>>7, because 147 exists. Then I press 2.
>>
>>No problem with vdr default skins.
>>
> 
> Thanks for the reply.  This is also exactly what I am seeing here.  If the
> channel numbers do not exist, then the numbers do not display.  There is also a
> similar problem I have with VDR where lets say I have two channels; 14 and
> channel 147.  I can not switch to channel 147 because by the time I press 1 and
> 4, VDR tunes to channel 14 before I can press 7.
> 

The following patch seems to fix the problem.  Although, I can't seem to figure
out why the default vdr skins have a trailing "-" sign as I enter the channel
keys, and this does not.  Please let me know if anyone spots a problem with this
 fix.

Best Regards,
  

Patch

--- text2skin/display.c.orig	2005-10-23 21:49:38.000000000 -0700
+++ text2skin/display.c	2005-10-23 21:51:13.000000000 -0700
@@ -118,7 +118,7 @@ 
 	case tChannelNumber:
 		return mChannel != NULL
 		       ? (cxType)ChannelNumber(mChannel, mNumber)
-		       : (cxType)false;
+		       : (cxType)mNumber;
 		
 	case tChannelName:
 		return mChannel != NULL