[commit] Another gdb_byte pass (correct use?)

[Paul Schlie]

Just to double check, as I thought the original intent of gdb_byte* was
an alternative to a void* to more accurately identify byte oriented data
not true char* or char[] strings? as if all 'char's are now 'gdb_byte's,
why bother, should have just left them alone?

(and compiled with unsigned char, or possibly get rid of the silly GCC
warning complaining about comparison of pointers to types differing in
signness, as the types are compatible.)