BinaryStream.ReadShort - should it be ReadInt8?

I’ve got some Windows-specific modules that I’m trying to integrate into a project and they make use of BinaryStreams. For many of the calls, an error occurs on the call to BinaryStream.ReadShort. I know it seems that converting these calls to ReadInt8 and be done with it is pretty obvious, but are there any potential gotcha’s waiting for me if I simply perform a global search/replace? Should I consider ReadUInt8 instead since these are mostly just handling 8bit data that really can’t be negative?

A Short is the same as an Int16.

Ah - so ReadUInt16. Weird since I could have sworn that those routines were reading bytes/characters. I guess I should review the old code again (this code dates back to REALbasic 4.5 times …).