On Thu, Jan 29, 2009 at 4:06 PM, Gregory Stark <stark@xxxxxxxxxxxxxxxx> wrote: > Gregory Stark <stark@xxxxxxxxxxxxxxxx> writes: > Ah, I understand how this fixes the problem. You were casting to unsigned > *int* not unsigned char so it was sign extending first and then overflowing. :) > It still seems to me if you put a few "unsigned" in variable declarations you > could remove piles upon piles of casts and make all of the code more readable. which is one of the main problems I see with that code, overall. -- GJ -- Sent via pgsql-general mailing list (pgsql-general@xxxxxxxxxxxxxx) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-general