[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gnuradio-commit
Subject:    [Commit-gnuradio] [gnuradio] 02/09: Fixing `Sdot` calculation in `fmdet_cf_impl` `work()`
From:       git () gnuradio ! org
Date:       2016-07-28 21:43:59
Message-ID: 20160728214359.B03B71A82A5 () hertz ! gnuradio ! org
[Download RAW message or body]

This is an automated email from the git hooks/post-receive script.

jcorgan pushed a commit to branch maint
in repository gnuradio.

commit 7c9f29519b709c432284fb6481842f03809ac594
Author: Ben Hilburn <ben.hilburn@ettus.com>
Date:   Thu Jul 14 10:18:21 2016 -0400

    Fixing `Sdot` calculation in `fmdet_cf_impl` `work()`
    
    Bug reported on-list. Current code actually results in the S1 terms simply
    canceling each other, whereas the second term should be S2.
---
 gr-analog/lib/fmdet_cf_impl.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gr-analog/lib/fmdet_cf_impl.cc b/gr-analog/lib/fmdet_cf_impl.cc
index 2c48ff0..38496d1 100644
--- a/gr-analog/lib/fmdet_cf_impl.cc
+++ b/gr-analog/lib/fmdet_cf_impl.cc
@@ -106,7 +106,7 @@ namespace gr {
       while(size-- > 0) {
 	S0 = *iptr++;
 
-	Sdot = d_scl * (-S0+d_8*S1-d_8*S1+S4);
+	Sdot = d_scl * (-S0+d_8*S1-d_8*S2+S4);
 
 	d_freq = (S2.real()*Sdot.imag()-S2.imag()*Sdot.real()) /
 	  (S2.real()*S2.real()+S2.imag()*S2.imag());

_______________________________________________
Commit-gnuradio mailing list
Commit-gnuradio@gnu.org
https://lists.gnu.org/mailman/listinfo/commit-gnuradio
[prev in list] [next in list] [prev in thread] [next in thread]