Fix qRcmd error code parsing
authorLuis Machado <luis.machado@arm.com>
Thu, 31 Mar 2022 15:45:53 +0000 (16:45 +0100)
committerLuis Machado <luis.machado@arm.com>
Tue, 5 Apr 2022 07:44:19 +0000 (08:44 +0100)
commitd5ce6f2dcacf9809fb7a29a69c4b98e0320c3c94
treedf1744e789b12e5b1fc4a55e55c9b48269fd0f72
parent27f9f649753b8c4bd4c40bde0f49d916c222a16a
Fix qRcmd error code parsing

Someone at IRC spotted a bug in qRcmd handling. This looks like an oversight
or it is that way for historical reasons.

The code in gdb/remote.c:remote_target::rcmd uses isdigit instead of
isxdigit. One could argue that we are expecting decimal numbers, but further
below we use fromhex ().

Update the function to use isxdigit instead and also update the documentation.

I see there are lots of other cases of undocumented number format for error
messages, mostly described as NN instead of nn. For now I'll just update
this particular function.
gdb/doc/gdb.texinfo
gdb/remote.c