Hi Christian,
On Sat, Jan 28, 2012 at 09:41:35PM +0100, Christian Vogel wrote:
Instead of calling schedule_timer() inside the callback (which causes list-reshuffling) just call rearm_timer(self,milliseconds) to have it re-execute after the specified time.
this may make sense from an efficiency point of view, but it is a deviation from the API we use in libosmocore. Which brings me to the next issue: The osmo_timer_* rename has not yet been applied to the timer implementation.
If other developers agree to introduce a 'rearm_timer', then we should also add it to osmo_timer_rearm() to libosmocore for consistency.
Also, it might be a good idea to verify that it actually is in the list, at least durinig DEBUG / development builds.
Regards, Harald