The RecordData class uses a signed int to store its size and implicitly converts it to size_t }}in a few places like calls to {{memcpy() and SharedBuffer::allocate().

Since there is no sentinel value for having a negative size, size should be stored as unsigned.

The outdated TODO in the file can be removed as well.