Gregory Stark <stark@enterprisedb.com> writes:
> I really think he should just change all the "unsigned int" into "unsigned
> char" and not do the type punning with pointer casts. That's just evil.
Oh, I see. That would work too, but I don't really see that it's a huge
improvement.
What *would* be an improvement IMHO is to declare the pointer as
unsigned char * in the first place ;-). However, that might result
in having to introduce a lot of casts elsewhere ...
regards, tom lane