Clean up overly paranoid checks in mac8.c
authorStephen Frost <sfrost@snowman.net>
Thu, 16 Mar 2017 03:23:28 +0000 (23:23 -0400)
committerStephen Frost <sfrost@snowman.net>
Thu, 16 Mar 2017 03:23:28 +0000 (23:23 -0400)
commit7821f7229c6e149046ee0dd8cab57928c4f86a37
treee3e952900b7e77ac29326cb09384235c231caa4d
parent6977b8b7f4dfb40896ff5e2175cad7fdbda862eb
Clean up overly paranoid checks in mac8.c

Andres' compiler points out, quite correctly, that there's no need for
some of the overly paranoid checks which were put into mac8.c.  Remove
those, as they're useless, add some comments and make a few other minor
improvements- reduce the size of hexlookup by making it a char array
instead of an int array, and pass in the ptr location directly instead
of making hex2_to_uchar re-calculate the location based off the offset
every time.
src/backend/utils/adt/mac8.c