@henningw commented on this pull request.
@petermarianF Thank you for integrating all the comments and sorry for the delay. I've
added two more comments, which are probably some minor things.
@@ -82,5 +83,7 @@ int update_security(udomain_t* _d,
security_type _t, security_t* _s, struct pcon
int update_temp_security(udomain_t*
_d, security_type _t, security_t* _s, struct pcontact* _c);
int preload_udomain(db1_con_t* _c, udomain_t* _d);
-
+int audit_usrloc_expired_pcontacts(udomain_t* _d);
+int db_load_pcontact(udomain_t* _d, str *_aor, int insert_cache, struct pcontact** _c,
pcontact_info_t* contact_info);
+int db_delete_presentityuri_from_pua(str *presentity_uri);
This probably can be removed as well, as there is no implementation.
@@ -301,7 +307,7 @@ int db_insert_pcontact(struct
pcontact* _c)
SET_PROPER_NULL_FLAG(_c->rinstance, values, LP_RINSTANCE_IDX);
SET_PROPER_NULL_FLAG(_c->rx_session_id, values, LP_RX_SESSION_ID_IDX);
- VAL_DOUBLE(GET_FIELD_IDX(values, LP_REG_STATE_IDX)) = _c->reg_state;
+ VAL_INT(GET_FIELD_IDX(values, LP_REG_STATE_IDX)) = _c->reg_state;
Why you are changing this from double to int?
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3279#pullrequestreview-1202820417
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/3279/review/1202820417(a)github.com>