@jcolp commented on this pull request.


> @@ -582,8 +582,10 @@ static void __dialog_sendpublish(
        }
 
        if(use_pubruri_avps && (refresh_pubruri_avps_flag > -1) && (request != 
NULL)
-                       && (request->flags
-                                       & (1U << (unsigned 
int)refresh_pubruri_avps_flag))) {
+               && (request

I believe this change is redundant. Line 584 already has "request != NULL" 
which is equivalent to this.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4795#pullrequestreview-4572133802
You are receiving this because you are subscribed to this thread.

Message ID: <kamailio/kamailio/pull/4795/review/[email protected]>
_______________________________________________
Kamailio - Development Mailing List -- [email protected]
To unsubscribe send an email to [email protected]
Important: keep the mailing list in the recipients, do not reply only to the 
sender!

Reply via email to