Fix namebreak and add additional unit tests

[Kileak]

Fix namebreak and add additional unit tests

Description/Motivation/Screenshots

fix #707

Using parse_address, when the user already specified a direct address like *0x40600 will result in dereferencing the address instead of just passing it to Breakpoint.

This patch replaces the default value for address, if none is given, with *$pc instead of $pc and removes parsing of the address completely, since user input will already be in the correct format for Breakpoint.

This way nb behaves more like default b command again

gef➤  nb testpc
Breakpoint 1 at 0x7ffff7ebde82: file ../sysdeps/unix/sysv/linux/read.c, line 26.
gef➤  nb testdirect *0x7ffff7ebde8a
Breakpoint 2 at 0x7ffff7ebde8a: file ../sysdeps/unix/sysv/linux/read.c, line 26.
gef➤  nb testdirect *0x7ffff7ebde8a+4
Breakpoint 3 at 0x7ffff7ebde8e: file ../sysdeps/unix/sysv/linux/read.c, line 26.
gef➤  nb testsymbol *read+32
Breakpoint 4 at 0x7ffff7ebde90: file ../sysdeps/unix/sysv/linux/read.c, line 26.

Kudos to @theguy147 for adding unit tests on namebreak. Added some additional tests, which will also check for direct address breakpoints.

How Has This Been Tested?

Architecture	Yes/No	Comments
x86-32	✔️
x86-64	✔️
ARM	✖️
AARCH64	✖️
MIPS	✖️
POWERPC	✖️
SPARC	✖️
RISC-V	✖️
make test	✔️

Checklist

• My PR was done against the dev branch, not master.
• My code follows the code style of this project.
• My change includes a change to the documentation, if required.
• My change adds tests as appropriate.
• I have read and agree to the CONTRIBUTING document.

[theguy147]

I agree, this is better than my fix!