[mythtv-commits] Ticket #1945: DVB-S/diseqc patch

MythTV mythtv at cvs.mythtv.org
Wed Jun 28 00:28:22 UTC 2006

#1945: DVB-S/diseqc patch
 Reporter:  yeasah at schwide.net  |        Owner:  danielk
     Type:  enhancement         |       Status:  new    
 Priority:  minor               |    Milestone:  0.20   
Component:  mythtv              |      Version:  head   
 Severity:  medium              |   Resolution:         
Comment (by yeasah at schwide.net):

 Replying to [comment:11 danielk]:
 > (In [10322]) Refs #1945. Initial commit of new DiSEqC code to diseqc
 > Yeasah, can you have a look over this and make sure it still works? I
 removed some code that looked like unrelated fixes. If they are related
 please generate a patch against this (w/explanation), if not generate a
 patch and ticket for each one against SVN head.

 I'll give it a test when I can, but I did notice a few things just going
 over the changeset, I think at the very least the last two will have to be
 corrected in one way or another before the stuff will work properly:

 * The always-use-auto-FEC thing is still in there, I never did take that
 out (sorry about that) -- line 749 of dvbchannel.cpp

 * The couple changes that you left out from siscan.cpp are indeed not
 related to the diseqc patch (DVB-S scanning fixes that leaked into the
 patch), so that's fine.

 * The changes to libmyth/settings.cpp+h are needed for the device tree
 editor to work properly. The change allows the myth listbox to have
 multiple strings with the same displayed value, which is needed for the
 GUI since the tree display is actually just an indented listbox -- and
 tree nodes can easily have the same name.

 * The code added to dvbchannel.cpp to delete the diseqc_tree object will
 cause crashes with the current implementation -- the lifetimes of tree
 objects is currently managed by the diseqcdevtrees class, so deleting the
 object in dvbchannel will leave an outstanding reference to a deleted
 object, which will either be reused or re-deleted at some future point.

 On an unrelated note, if it's easy to take the new rotor position status
 signal monitor value and stick it in the OSD, that would be a really nice
 addition to this patch. I have a pretty rough understanding of the signal
 monitor code, so I didn't want to do much more to that stuff than was
 necessary in the patch.

Ticket URL: <http://cvs.mythtv.org/trac/ticket/1945>
MythTV <http://www.mythtv.org/>

More information about the mythtv-commits mailing list