commit 7a266775af9369e22065920a5c053fd78c432178
author Michal Vasko <mvasko@cesnet.cz> Tue Jan 23 11:02:38 2024 +0100
committer Michal Vasko <mvasko@cesnet.cz> Mon Feb 26 13:15:26 2024 +0100
tree dab9570c66ca7ee6f3c9ba47c9b953d7790398ff
parent 2c1f66d35a48ddfdc6984682f74de525b274d0a2

libyang REFACTOR major logging updates

Structured error information instead of a
single error message.

src/parser_xml.c

diff --git a/src/parser_xml.c b/src/parser_xml.c
index 6978475..02c6be6 100644
--- a/src/parser_xml.c
+++ b/src/parser_xml.c
@@ -75,7 +75,7 @@
 
     *meta = NULL;
 
-    LOG_LOCSET(sparent, NULL, NULL, NULL);
+    LOG_LOCSET(sparent, NULL);
 
     /* check for NETCONF filter unqualified attributes */
     if (!strcmp(sparent->module->name, "notifications")) {
@@ -165,7 +165,7 @@
     }
 
 cleanup:
-    LOG_LOCBACK(1, 0, 0, 0);
+    LOG_LOCBACK(1, 0);
     if (ret) {
         lyd_free_meta_siblings(*meta);
         *meta = NULL;
@@ -672,7 +672,7 @@
     LY_CHECK_GOTO(rc, cleanup);
 
     assert(*node);
-    LOG_LOCSET(NULL, *node, NULL, NULL);
+    LOG_LOCSET(NULL, *node);
 
     /* parser next */
     rc = lyxml_ctx_next(xmlctx);
@@ -709,7 +709,7 @@
 
 cleanup:
     if (*node) {
-        LOG_LOCBACK(0, 1, 0, 0);
+        LOG_LOCBACK(0, 1);
     }
     ly_free_prefix_data(format, val_prefix_data);
     if (dynamic) {
@@ -747,7 +747,7 @@
     LY_DPARSER_ERR_GOTO(r, rc = r, lydctx, cleanup);
 
     if (*node) {
-        LOG_LOCSET(NULL, *node, NULL, NULL);
+        LOG_LOCSET(NULL, *node);
     }
 
     if (*node && parent && (snode->flags & LYS_KEY)) {
@@ -778,7 +778,7 @@
 
 cleanup:
     if (*node) {
-        LOG_LOCBACK(0, 1, 0, 0);
+        LOG_LOCBACK(0, 1);
     }
     if (rc && (!(lydctx->val_opts & LYD_VALIDATE_MULTI_ERROR) || (rc != LY_EVALID))) {
         lyd_free_tree(*node);
@@ -819,7 +819,7 @@
     LY_CHECK_GOTO(rc, cleanup);
 
     assert(*node);
-    LOG_LOCSET(NULL, *node, NULL, NULL);
+    LOG_LOCSET(NULL, *node);
 
     /* parser next */
     rc = lyxml_ctx_next(xmlctx);
@@ -863,7 +863,7 @@
 
 cleanup:
     if (*node) {
-        LOG_LOCBACK(0, 1, 0, 0);
+        LOG_LOCBACK(0, 1);
     }
     lydctx->parse_opts = prev_parse_opts;
     if (rc && ((*node && !(*node)->hash) || !(lydctx->val_opts & LYD_VALIDATE_MULTI_ERROR) || (rc != LY_EVALID))) {
@@ -923,7 +923,7 @@
         LY_CHECK_ERR_GOTO(r, rc = r, cleanup);
 
         assert(*node);
-        LOG_LOCSET(NULL, *node, NULL, NULL);
+        LOG_LOCSET(NULL, *node);
         log_node = 1;
 
         /* parser next */
@@ -948,7 +948,7 @@
 
 cleanup:
     if (log_node) {
-        LOG_LOCBACK(0, 1, 0, 0);
+        LOG_LOCBACK(0, 1);
     }
     lydctx->parse_opts = prev_parse_opts;
     lydctx->int_opts = prev_int_opts;