jchavanton commented on this pull request.


>       if (is_log_mc_on(req)) {
                env_set_text( ACC_MISSED, ACC_MISSED_LEN);
                acc_log_request( req );
-               flags_to_reset |= log_missed_flag;
+               flags_to_reset |= 1 << log_missed_flag;

I wonder why this was not creating more problem in the past the flags variable 
where containing "bit position", this is one point that the reviewer can help 
to confirm. (I may triple check that this is always true)

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/1674#pullrequestreview-164008683
_______________________________________________
Kamailio (SER) - Development Mailing List
[email protected]
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to