Fix search range notation in search herald

[kazarmy]

Your checklist for this pull request

• I've read the guidelines for contributing to this repository.
• I made sure to follow the project's coding style.
• I've documented every RZ_API function and struct this PR changes.
• I've added tests that prove my changes are effective (required for changes to RZ_API).
• I've updated the Rizin book with the relevant information (if needed).

Detailed description

From the definition of rz_itv_end():

rizin/librz/include/rz_util/rz_itv.h

Lines 42 to 44 in 2c1a07d

	static inline ut64 rz_itv_end(RzInterval itv) {
	return itv.addr + itv.size;
	}

the address returned by it is one byte beyond the interval and so this pr changes the range notation used in the search herald to reflect that.

Test plan

All builds are green.

Closing issues

...

[wargio]

I fully agree on this

[Rot127]

Sorry, pressed the button too quickly again. Can you please add this to the struct doxygen (in a more readable manner with \brief and all)? This is where people will look first.

[wargio]

This should be explained in the book

[kazarmy]

Can you please add this to the struct doxygen ...

Ok 51ef7eb

[kazarmy]

This should be explained in the book

I have looked at the book and noticed that

1. The search hits as currently presented could be better.
2. There can be inconsistencies between / lib and /e /lib/ 🤦.

[Rot127]

The search hits as currently presented could be better.

This is on the list. Once the PR is reviewed and done I will rewrite the whole chapter. Also covering edge cases and such.

There can be inconsistencies between / lib and /e /lib/

Also let's wait for refactor. Since it takes regex now.