From 38b293e74db6631873d5dfd74cf731eb7b76737d Mon Sep 17 00:00:00 2001 From: Andrew Gallant Date: Sun, 11 Aug 2013 20:54:15 -0400 Subject: Padding on a list is on the length of the list. There was a bug where padding was being computed on each element of the list. Close #5. --- nexgb/record/record.go | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) (limited to 'nexgb/record/record.go') diff --git a/nexgb/record/record.go b/nexgb/record/record.go index 8cb495e..2644f13 100644 --- a/nexgb/record/record.go +++ b/nexgb/record/record.go @@ -143,9 +143,9 @@ func ClientInfoListBytes(buf []byte, list []ClientInfo) int { for _, item := range list { structBytes = item.Bytes() copy(buf[b:], structBytes) - b += xgb.Pad(len(structBytes)) + b += len(structBytes) } - return b + return xgb.Pad(b) } // ClientInfoListSize computes the size (bytes) of a list of ClientInfo values. @@ -232,9 +232,9 @@ func ExtRangeListBytes(buf []byte, list []ExtRange) int { for _, item := range list { structBytes = item.Bytes() copy(buf[b:], structBytes) - b += xgb.Pad(len(structBytes)) + b += len(structBytes) } - return b + return xgb.Pad(b) } const ( @@ -378,9 +378,9 @@ func RangeListBytes(buf []byte, list []Range) int { for _, item := range list { structBytes = item.Bytes() copy(buf[b:], structBytes) - b += xgb.Pad(len(structBytes)) + b += len(structBytes) } - return b + return xgb.Pad(b) } type Range16 struct { @@ -432,9 +432,9 @@ func Range16ListBytes(buf []byte, list []Range16) int { for _, item := range list { structBytes = item.Bytes() copy(buf[b:], structBytes) - b += xgb.Pad(len(structBytes)) + b += len(structBytes) } - return b + return xgb.Pad(b) } type Range8 struct { @@ -486,9 +486,9 @@ func Range8ListBytes(buf []byte, list []Range8) int { for _, item := range list { structBytes = item.Bytes() copy(buf[b:], structBytes) - b += xgb.Pad(len(structBytes)) + b += len(structBytes) } - return b + return xgb.Pad(b) } // Skipping definition for base type 'Bool' -- cgit v1.2.3