roll back r10806 because 1) all callers to this function have names which are MAX_FILENAME_LEN in size; and 2) the memset already sets all characters to 0, so there's no need to add a null terminator

git-svn-id: svn://svn.icculus.org/fs2open@10857 387891d4-d844-0410-90c0-e4c51a9137d3
fix a bug introduced by r10837: 0 is a valid path index

git-svn-id: svn://svn.icculus.org/fs2open@10858 387891d4-d844-0410-90c0-e4c51a9137d3
fix a bug introduced by r10852: MAX_OBJECTS is not a valid object index

git-svn-id: svn://svn.icculus.org/fs2open@10859 387891d4-d844-0410-90c0-e4c51a9137d3
Fix a range of issue identified via clang warnings

git-svn-id: svn://svn.icculus.org/fs2open@10860 387891d4-d844-0410-90c0-e4c51a9137d3
Address Clang warnings are overloaded virtual functions in UI_GADET. Also opportunity to simplify code by making the most of inherited polymorphism. Docs cleanup

git-svn-id: svn://svn.icculus.org/fs2open@10861 387891d4-d844-0410-90c0-e4c51a9137d3
Quieten a number of warnings related to UI_USERBOX without a declaration yet. Added TODO.

git-svn-id: svn://svn.icculus.org/fs2open@10862 387891d4-d844-0410-90c0-e4c51a9137d3