From d26a35d10b9e2f808b0e4880d5b5d1a2963258e9 Mon Sep 17 00:00:00 2001 From: Michael Sevakis Date: Tue, 8 May 2012 22:47:51 -0400 Subject: [PATCH] Tweak dsp_format_change_process (default format handler). Just stop searching if the entry is found (as it should have been). Change-Id: Id968694e825282d58c8ca4a7789c236f98643a5f --- lib/rbcodec/dsp/dsp_core.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/lib/rbcodec/dsp/dsp_core.c b/lib/rbcodec/dsp/dsp_core.c index 175b9c1c64..84a23a4c83 100644 --- a/lib/rbcodec/dsp/dsp_core.c +++ b/lib/rbcodec/dsp/dsp_core.c @@ -128,13 +128,15 @@ static void dsp_format_change_process(struct dsp_proc_entry *this, /* We don't keep back references to the DSP, so just search for it */ struct dsp_config *dsp; - for (int i = 0; (dsp = dsp_get_config(i)); i++) + for (unsigned int i = 0; (dsp = dsp_get_config(i)); i++) { - struct dsp_proc_slot *slot = find_proc_slot(dsp, id); - /* Found one with the id, check if it's this one */ - if (&slot->proc_entry == this && dsp_proc_active(dsp, id)) + /* Found one with the id, check if it's this one + (NULL return doesn't matter) */ + if (&find_proc_slot(dsp, id)->proc_entry == this) { - dsp_proc_call(this, buf_p, 0); + if (dsp_proc_active(dsp, id)) + dsp_proc_call(this, buf_p, 0); + break; } }