Limit bandmap_lookup to the list of displayed spots

[dl1jbe]

No description provided.

[dl1jbe]

A quick hack to limit the search in 'bandmap_lookup()' to the displayed list of spots. May need some more love.

[zcsahok]

Looks good, thanks. Not sure whether the start/stop variables need to be protected by the mutex as grabbing happens in the main thread. Also keeping 'i < spots->len' would not hurt.

[dl1jbe]

Not sure whether the start/stop variables need to be protected by the mutex as grabbing happens in the main thread.

Yes, 'spots' array and the variables needs to stay in sync. But we can not just wrap the whole bandmap_show() by the mutex as it is already used in filter_spots() and the mutex can not be used recursive.

I will look into the problem - just give me time till Sunday. There are some more minor changes needed in bandmap.c anyway.

[dl1jbe]

Besides some refactoring of the range calculation and the final spot display it should be correct now.
The mutex covers now the built of the spots array AND the calculation of the display range.

[zcsahok]

Looks good. There are just 2 typos (frequencyi, capsi; using vi?).

When we at it, could we maybe have a look at the selection of bandmap's current frequency? Not sure how it shall work, but it's asymmetric now.

Example:

freq=3543.5, it is marked as different from the spot's frequency (3543.7), fine:

freq=3543.6, we are 100 Hz off and it is "rounded up" to the spot:

freq=3543.7, we are spot on:

freq=3543.8, we are again 100 Hz off, but no rounding takes place: