Module: kamailio
Branch: master
Commit: 67f71d1b7c44a7d3ef1ba2917272e2ba36608cd7
URL: 
https://github.com/kamailio/kamailio/commit/67f71d1b7c44a7d3ef1ba2917272e2ba36608cd7

Author: Daniel-Constantin Mierla <mico...@gmail.com>
Committer: Daniel-Constantin Mierla <mico...@gmail.com>
Date: 2024-01-20T12:37:12+01:00

lib/xcap: reviewed and removed old comments and disabled code

---

Modified: src/lib/xcap/common_policy.c
Modified: src/lib/xcap/parse_common_rules.c
Modified: src/lib/xcap/resource_list.c
Modified: src/lib/xcap/xcap_client.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/67f71d1b7c44a7d3ef1ba2917272e2ba36608cd7.diff
Patch: 
https://github.com/kamailio/kamailio/commit/67f71d1b7c44a7d3ef1ba2917272e2ba36608cd7.patch

---

diff --git a/src/lib/xcap/common_policy.c b/src/lib/xcap/common_policy.c
index a7d33d95d5d..0f887aa7ce7 100644
--- a/src/lib/xcap/common_policy.c
+++ b/src/lib/xcap/common_policy.c
@@ -175,7 +175,7 @@ int is_rule_for_uri(cp_rule_t *rule, const str_t *uri)
        if(!rule)
                return 0;
        if(!rule->conditions)
-               return 1; /* FIXME: ??? */
+               return 1;
        id = rule->conditions->identity;
        if(!id)
                return 0;
diff --git a/src/lib/xcap/parse_common_rules.c 
b/src/lib/xcap/parse_common_rules.c
index 0e46ace7161..ca3f4599c06 100644
--- a/src/lib/xcap/parse_common_rules.c
+++ b/src/lib/xcap/parse_common_rules.c
@@ -211,13 +211,11 @@ static int read_conditions(xmlNode *cn, cp_conditions_t 
**dst)
        while(n) {
                if(n->type == XML_ELEMENT_NODE) {
                        if(cmp_node(n, "validity", common_policy_ns) >= 0) {
-                               /* FIXME: free existing validity */
                                res = read_validity(n, &(*dst)->validity);
                                if(res != 0)
                                        break;
                        } else {
                                if(cmp_node(n, "identity", common_policy_ns) >= 
0) {
-                                       /* FIXME: free existing identity */
                                        res = read_identity(n, 
&(*dst)->identity);
                                        if(res != 0)
                                                break;
diff --git a/src/lib/xcap/resource_list.c b/src/lib/xcap/resource_list.c
index cb7d111057f..928c133df51 100644
--- a/src/lib/xcap/resource_list.c
+++ b/src/lib/xcap/resource_list.c
@@ -169,9 +169,9 @@ void free_traversed_list(traversed_list_t *list)
 
 /* ------- helper functions (doing flat list) ------- */
 
+/* absolute uri from ref (RFC 3986, section 5.2) */
 static char *relative2absolute_uri(const str_t *xcap_root, const char 
*relative)
 {
-       /* FIXME: do absolute uri from ref (RFC 3986, section 5.2) */
        int len;
        int root_len = 0;
        int rel_len = 0;
diff --git a/src/lib/xcap/xcap_client.c b/src/lib/xcap/xcap_client.c
index c633f3c7f5f..2fe6e41d1d6 100644
--- a/src/lib/xcap/xcap_client.c
+++ b/src/lib/xcap/xcap_client.c
@@ -284,13 +284,8 @@ int xcap_query(
                /* follow redirects (needed for apache mod_speling - case 
insesitive names) */
                curl_easy_setopt(handle, CURLOPT_FOLLOWLOCATION, 1);
 
-               /*      curl_easy_setopt(handle, CURLOPT_TCP_NODELAY, 1);
-               curl_easy_setopt(handle, CURLOPT_CONNECTTIMEOUT, 10);*/
-
                /* Accept headers */
-
                res = curl_easy_perform(handle);
-               /* curl_easy_cleanup(handle); */ /* FIXME: experimental */
        } else
                ERROR_LOG("can't initialize curl handle\n");
        if(res == 0) {

_______________________________________________
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org

Reply via email to