String and shape utils clean up

[agibsonccc]

What changes were proposed in this pull request?

99% compiler warnings changes
Removes unused methods in string/shape utils

How was this patch tested?

This is part of a series of PRs that were created from running benchmarks and troubleshooting some very mysterious segfaults via race conditions and undefined behavior.

Quick checklist

The following checklist helps ensure your PR is complete:

• [X ] Eclipse Contributor Agreement signed, and signed commits - see IP Requirements page for details
• [ X] Reviewed the Contributing Guidelines and followed the steps within.
• [ X] Created tests for any significant new code additions.
• [ X] Relevant tests for your changes are passing.