Fix search range notation in search herald (#4871)

* Fix search range notation in search herald
* Add doxygen to RzInterval

librz/core/cmd/cmd_search.c

@@ -1284,7 +1284,7 @@ static void do_string_search(RzCore *core, RzInterval search_itv, struct search_
 					const char *bytestr = lenstr > 1 ? "bytes" : "byte";
 					eprintf(" %d %s", kw ? kw->keyword_length : 0, bytestr);
 				}
-				eprintf(" in [0x%" PFMT64x "-0x%" PFMT64x "]\n", itv.addr, rz_itv_end(itv));
+				eprintf(" in [0x%" PFMT64x ",0x%" PFMT64x ")\n", itv.addr, rz_itv_end(itv));
 			}
 			if (!core->search->bckwrds) {
 				RzListIter *it;

librz/include/rz_util/rz_itv.h

@@ -7,13 +7,16 @@
 extern "C" {
 #endif
 
-// An interval in 64-bit address space which is aware of address space wraparound
-// Precondition: 0 <= size < 2**64 and addr + size <= 2**64
-// range is [], [10, 5) => 10 <= x < (10 + 5)
+/**
+ * \brief An interval in 64-bit address space which is aware of address space wraparound.
+ *
+ * Precondition: 0 <= size < 2**64 and addr + size <= 2**64
+ * Interval range is [addr, addr + size)
+ * e.g. with addr = 10 and size = 5, interval range is [10, 15) where 10 <= x < (10 + 5).
+ */
 typedef struct rz_interval_t {
-	// public:
-	ut64 addr;
-	ut64 size;
+	ut64 addr; ///< Start address of the interval.
+	ut64 size; ///< Size of the interval in bytes.
 } RzInterval;
 
 typedef RzInterval rz_itv_t;

test/db/cmd/regexp

@@ -76,15 +76,15 @@ EXPECT=<<EOF
 0x00000010 hit4_0 .abcdbcccde.
 EOF
 EXPECT_ERR=<<EOF
-Searching in [0x0-0x200]
+Searching in [0x0,0x200)
 hits: 2
-Searching in [0x0-0x200]
+Searching in [0x0,0x200)
 hits: 0
-Searching in [0x0-0x200]
+Searching in [0x0,0x200)
 hits: 2
-Searching in [0x0-0x200]
+Searching in [0x0,0x200)
 hits: 1
-Searching in [0x0-0x200]
+Searching in [0x0,0x200)
 hits: 1
 EOF
 RUN
@@ -107,9 +107,9 @@ EXPECT=<<EOF
 0x000001fe hit1_0 .bd.
 EOF
 EXPECT_ERR=<<EOF
-Searching in [0x0-0x200]
+Searching in [0x0,0x200)
 hits: 2
-Searching in [0x0-0x200]
+Searching in [0x0,0x200)
 hits: 1
 EOF
 RUN