summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRafaël Carré <rafael.carre@gmail.com>2010-09-14 06:34:01 +0000
committerRafaël Carré <rafael.carre@gmail.com>2010-09-14 06:34:01 +0000
commit0928cdf074c8991f470fa0d96e6d4f828998b643 (patch)
treef29e57c62cb894c5fbcebd60f4ddad3043d78c85
parentcba9d259b05a88302a78669cba0953ba0c98274d (diff)
downloadrockbox-0928cdf074c8991f470fa0d96e6d4f828998b643.tar.gz
rockbox-0928cdf074c8991f470fa0d96e6d4f828998b643.tar.bz2
rockbox-0928cdf074c8991f470fa0d96e6d4f828998b643.zip
pitch_detector: don't use a struct with only one member
use fixed point type (fixed == int32_t) directly git-svn-id: svn://svn.rockbox.org/rockbox/trunk@28072 a1c6a512-1295-4272-9138-f99709370657
-rw-r--r--apps/plugins/pitch_detector.c123
1 files changed, 58 insertions, 65 deletions
diff --git a/apps/plugins/pitch_detector.c b/apps/plugins/pitch_detector.c
index 45184031b2..be47823e0d 100644
--- a/apps/plugins/pitch_detector.c
+++ b/apps/plugins/pitch_detector.c
@@ -70,45 +70,38 @@
/* Some fixed point calculation stuff */
-typedef int32_t fixed_data;
-typedef struct
-{
- fixed_data a;
-} fixed;
+typedef int32_t fixed;
#define FIXED_PRECISION 18
#define FP_MAX ((fixed) {0x7fffffff})
#define FP_MIN ((fixed) {-0x80000000})
-#define int2fixed(x) ((fixed){(x) << FIXED_PRECISION})
-#define int2mantissa(x) ((fixed){x})
-#define fixed2int(x) ((int)((x).a >> FIXED_PRECISION))
-#define fixed2float(x) (((float)(x).a) / ((float)(1 << FIXED_PRECISION)))
-
-/* cast in tables confuse gcc -std=gnu99 */
-#define float2fixed_decl(x) {(fixed_data)(x * (float)(1 << FIXED_PRECISION))}
-#define float2fixed(x) ((fixed)float2fixed_decl(x))
+#define int2fixed(x) ((fixed)((x) << FIXED_PRECISION))
+#define int2mantissa(x) ((fixed)(x))
+#define fixed2int(x) ((int)((x) >> FIXED_PRECISION))
+#define fixed2float(x) (((float)(x)) / ((float)(1 << FIXED_PRECISION)))
+#define float2fixed(x) ((fixed)(x * (float)(1 << FIXED_PRECISION)))
/* I adapted these ones from the Rockbox fixed point library */
#define fp_mul(x, y) \
- ((fixed){(((int64_t)((x).a)) * ((int64_t)((y).a))) >> (FIXED_PRECISION)})
+ ((fixed)((((int64_t)((x))) * ((int64_t)((y)))) >> (FIXED_PRECISION)))
#define fp_div(x, y) \
- ((fixed){(((int64_t)((x).a)) << (FIXED_PRECISION)) / ((int64_t)((y).a))})
+ ((fixed)((((int64_t)((x))) << (FIXED_PRECISION)) / ((int64_t)((y)))))
/* Operators for fixed point */
-#define fp_add(x, y) ((fixed){(x).a + (y).a})
-#define fp_sub(x, y) ((fixed){(x).a - (y).a})
-#define fp_shl(x, y) ((fixed){(x).a << y})
-#define fp_shr(x, y) ((fixed){(x).a >> y})
-#define fp_neg(x) ((fixed){-(x).a})
-#define fp_gt(x, y) ((x).a > (y).a)
-#define fp_gte(x, y) ((x).a >= (y).a)
-#define fp_lt(x, y) ((x).a < (y).a)
-#define fp_lte(x, y) ((x).a <= (y).a)
+#define fp_add(x, y) ((fixed)((x) + (y)))
+#define fp_sub(x, y) ((fixed)((x) - (y)))
+#define fp_shl(x, y) ((fixed)((x) << (y)))
+#define fp_shr(x, y) ((fixed)((x) >> (y)))
+#define fp_neg(x) ((fixed)(-(x)))
+#define fp_gt(x, y) ((x) > (y))
+#define fp_gte(x, y) ((x) >= (y))
+#define fp_lt(x, y) ((x) < (y))
+#define fp_lte(x, y) ((x) <= (y))
#define fp_sqr(x) fp_mul((x), (x))
-#define fp_equal(x, y) ((x).a == (y).a)
+#define fp_equal(x, y) ((x) == (y))
#define fp_round(x) (fixed2int(fp_add((x), float2fixed(0.5))))
-#define fp_data(x) ((x).a)
+#define fp_data(x) (x)
#define fp_frac(x) (fp_sub((x), int2fixed(fixed2int(x))))
-#define FP_ZERO ((fixed){0})
-#define FP_LOW ((fixed){2})
+#define FP_ZERO ((fixed)0)
+#define FP_LOW ((fixed)2)
/* Some defines for converting between period and frequency */
@@ -152,20 +145,20 @@ typedef struct
#define DEFAULT_YIN_THRESHOLD 5 /* 0.10 */
static const fixed yin_threshold_table[] IDATA_ATTR =
{
- float2fixed_decl(0.01),
- float2fixed_decl(0.02),
- float2fixed_decl(0.03),
- float2fixed_decl(0.04),
- float2fixed_decl(0.05),
- float2fixed_decl(0.10),
- float2fixed_decl(0.15),
- float2fixed_decl(0.20),
- float2fixed_decl(0.25),
- float2fixed_decl(0.30),
- float2fixed_decl(0.35),
- float2fixed_decl(0.40),
- float2fixed_decl(0.45),
- float2fixed_decl(0.50),
+ float2fixed(0.01),
+ float2fixed(0.02),
+ float2fixed(0.03),
+ float2fixed(0.04),
+ float2fixed(0.05),
+ float2fixed(0.10),
+ float2fixed(0.15),
+ float2fixed(0.20),
+ float2fixed(0.25),
+ float2fixed(0.30),
+ float2fixed(0.35),
+ float2fixed(0.40),
+ float2fixed(0.45),
+ float2fixed(0.50),
};
/* Structure for the reference frequency (frequency of A)
@@ -180,17 +173,17 @@ static const struct
const fixed logratio; /* log2(factor) */
} freq_A[] =
{
- {435, float2fixed_decl(1.011363636), float2fixed_decl( 0.016301812)},
- {436, float2fixed_decl(1.009090909), float2fixed_decl( 0.013056153)},
- {437, float2fixed_decl(1.006818182), float2fixed_decl( 0.009803175)},
- {438, float2fixed_decl(1.004545455), float2fixed_decl( 0.006542846)},
- {439, float2fixed_decl(1.002272727), float2fixed_decl( 0.003275132)},
- {440, float2fixed_decl(1.000000000), float2fixed_decl( 0.000000000)},
- {441, float2fixed_decl(0.997727273), float2fixed_decl(-0.003282584)},
- {442, float2fixed_decl(0.995454545), float2fixed_decl(-0.006572654)},
- {443, float2fixed_decl(0.993181818), float2fixed_decl(-0.009870244)},
- {444, float2fixed_decl(0.990909091), float2fixed_decl(-0.013175389)},
- {445, float2fixed_decl(0.988636364), float2fixed_decl(-0.016488123)},
+ {435, float2fixed(1.011363636), float2fixed( 0.016301812)},
+ {436, float2fixed(1.009090909), float2fixed( 0.013056153)},
+ {437, float2fixed(1.006818182), float2fixed( 0.009803175)},
+ {438, float2fixed(1.004545455), float2fixed( 0.006542846)},
+ {439, float2fixed(1.002272727), float2fixed( 0.003275132)},
+ {440, float2fixed(1.000000000), float2fixed( 0.000000000)},
+ {441, float2fixed(0.997727273), float2fixed(-0.003282584)},
+ {442, float2fixed(0.995454545), float2fixed(-0.006572654)},
+ {443, float2fixed(0.993181818), float2fixed(-0.009870244)},
+ {444, float2fixed(0.990909091), float2fixed(-0.013175389)},
+ {445, float2fixed(0.988636364), float2fixed(-0.016488123)},
};
/* Index of the entry for 440 Hz in the table (default frequency for A) */
@@ -268,18 +261,18 @@ static const struct
const fixed logfreq; /* log2(frequency) */
} notes[] =
{
- {"A" , float2fixed_decl(440.0000000f), float2fixed_decl(8.781359714f)},
- {"A#", float2fixed_decl(466.1637615f), float2fixed_decl(8.864693047f)},
- {"B" , float2fixed_decl(493.8833013f), float2fixed_decl(8.948026380f)},
- {"C" , float2fixed_decl(523.2511306f), float2fixed_decl(9.031359714f)},
- {"C#", float2fixed_decl(554.3652620f), float2fixed_decl(9.114693047f)},
- {"D" , float2fixed_decl(587.3295358f), float2fixed_decl(9.198026380f)},
- {"D#", float2fixed_decl(622.2539674f), float2fixed_decl(9.281359714f)},
- {"E" , float2fixed_decl(659.2551138f), float2fixed_decl(9.364693047f)},
- {"F" , float2fixed_decl(698.4564629f), float2fixed_decl(9.448026380f)},
- {"F#", float2fixed_decl(739.9888454f), float2fixed_decl(9.531359714f)},
- {"G" , float2fixed_decl(783.9908720f), float2fixed_decl(9.614693047f)},
- {"G#", float2fixed_decl(830.6093952f), float2fixed_decl(9.698026380f)},
+ {"A" , float2fixed(440.0000000f), float2fixed(8.781359714f)},
+ {"A#", float2fixed(466.1637615f), float2fixed(8.864693047f)},
+ {"B" , float2fixed(493.8833013f), float2fixed(8.948026380f)},
+ {"C" , float2fixed(523.2511306f), float2fixed(9.031359714f)},
+ {"C#", float2fixed(554.3652620f), float2fixed(9.114693047f)},
+ {"D" , float2fixed(587.3295358f), float2fixed(9.198026380f)},
+ {"D#", float2fixed(622.2539674f), float2fixed(9.281359714f)},
+ {"E" , float2fixed(659.2551138f), float2fixed(9.364693047f)},
+ {"F" , float2fixed(698.4564629f), float2fixed(9.448026380f)},
+ {"F#", float2fixed(739.9888454f), float2fixed(9.531359714f)},
+ {"G" , float2fixed(783.9908720f), float2fixed(9.614693047f)},
+ {"G#", float2fixed(830.6093952f), float2fixed(9.698026380f)},
};
/* GUI */