[Discuss-gnuradio] gri_control_loop::set_frequency() Possible Error?

[Frederick Lee]

Hi, 

I was looking at the gri_control_loop.cc file and notices the set_frequency() function seemed a little odd. Here's what it looked like.

void
gri_control_loop::set_frequency(float freq)
{
  if(freq > d_max_freq)
    d_freq = d_min_freq;
  else if(freq < d_min_freq)
    d_freq = d_max_freq;
  else
    d_freq = freq;
}

This doesn't seem logical to me. Shouldn't the it be "d_freq = d_max_freq" in the if statement and "d_freq = d_min_freq" in the else if statement?
The frequency_limit() function has it the way I stated

void
gri_control_loop::frequency_limit()
{
  if (d_freq > d_max_freq)
    d_freq = d_max_freq;
  else if (d_freq < d_min_freq)
    d_freq = d_min_freq;
}

Regards,

Frederick