On Thu, 22 Jun 2006, James Cameron wrote:
> On Wed, Jun 21, 2006 at 02:34:30AM -0700, William Balcerski wrote:
> > Hmm I guess what I was trying to get at was, if fuel were ever larger than
> > an unsigned int could hold, there would be problems in numerous places,
> > especially client side as they would not be able to accept fuel as a long
> > int.  So the margin check would have no impact.
>
> So what?  It's still useful, as it would prevent short packets from
> being used in the event a larger fuel was used.  In the SP_YOU packet it
> is sent as a LONG.

I think Bill does not understand the one packet uses a short and the other
uses a long.
>
> The long switch statement determines if a packet is sent by UDP or TCP.
> See the gwrite() call on line 786 for UDP, and the call on line 821 for
> TCP.

Bill forgot to update this switch statement in his patch.