Hi Henning, in the end we where calling if_update_stat twice.
This code is only called when retrieving dialog from the DB.

Yes my comment was about removing the optimized version and keeping the simpler one, if_update_stat is is a macro calling several inline functions.
One way of calling it was to using a counter when in order to call if_update_stat only once.
Then as second call to if_update_stat was added in b0cd09 probably because the optimization with the counter was obfuscating things slightly.

I think the optimization are not needed the cost per dialog seems very low.
But we could still select to keep the optimized version if we want, knowing that since 2016 both version where used anyway

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.