[mythtv] [PATCH] Channel preset hack

Ben Bucksch linux.news at bucksch.org
Fri May 16 23:13:57 EDT 2003


  ramon.roca at xcombo.com wrote:

>Would not be easier to just use chanid
>
The chanid is used to connect tables in the database. You shouldn't use 
these primary keys for other purposes, and esp they should never ever 
change. If you change the chanid, all the entries referring to them 
(records etc.) go invalid and you have to fix them all, rename the 
recording files etc.. Because it is likely that the user may want to 
change the preset arrangement, this is not a good route IMO.

>I haven't looked to the source for this yet, but seems less agressive since does not require additional columns to the database, maybe that will require a more intrussive patch to the code?
>
Yes. The channum is used all over the code. It (not the chanid) is 
passed around as ID for the channel and then shown in many places in the 
UI. I'd have to touch the OSD, the remote control code, the ChanUp/Down 
code, the EPG, the list of recordings, all themes, MythWeb etc.etc. to 
achieve the same result.

The difference is a 10 line patch against a 1000 line patch or something 
in that order.

My patch is just a "works for me, for now, HTH others" :).



More information about the mythtv-dev mailing list