[mythtv] CVS : External channel changer executes twice

[Mark A. Mankins]

I just switched to the cvs version of mythtv.  One problem I've noticed is that the cvs version runs the external channel changer twice prior to recording a program.  Since the external channel changer command is run in the background, what ends up happening is that two channel changer commands end up running at essentially the same time.  So instead of changing the channel to 123, what gets sent to the cable box is something like 121233.  I traced the problem to the following piece of code.  The patch below fixed the problem for me, but I'm sure the code was there for a reason, so can someone look at this an apply a proper patch?

Thanks!

Mark

diff -C3 -r1.148 tv_rec.cpp
*** libs/libmythtv/tv_rec.cpp   21 May 2004 03:39:58 -0000      1.148
--- libs/libmythtv/tv_rec.cpp   23 May 2004 12:06:31 -0000
***************
*** 496,502 ****
              bool success = channel->Open();
              if (success)
              {
!                 success = channel->SetChannelByString(channel->GetCurrentName());
                  if (!success)
                  {
                      VERBOSE(VB_IMPORTANT, "Signal level too low?");
--- 496,502 ----
              bool success = channel->Open();
              if (success)
              {
!                 //success = channel->SetChannelByString(channel->GetCurrentName());
                  if (!success)
                  {
                      VERBOSE(VB_IMPORTANT, "Signal level too low?");