changed char type to uint8_t and fixed std::string_view type cast werror

[alexchunlin]

Description

These were the changes that made the compiler stop screaming

Motivation and Context

Closes #7

How has this been tested?

Demo builds and runs on windows 11 && UE5.2.1

Screenshots (if appropriate, e.g. schematic, board, console logs, lab pictures):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation Update
• Software change

Checklist:

• My change requires a change to the documentation.
• I have added / updated the documentation related to this change via either README or WIKI

Software

• I have added tests to cover my changes.
• All new and existing tests passed.
• My code follows the code style of this project.

[finger563]

Looks good, thanks! I had changed some of the lower level (RtpPacket) containers over to uint8_t but hadn't needed to change the higher level stuff when compiling with clang. Glad you got it working / tested on Windows 👍