From 9492662dae0651fd4d6698d35b58ade7300e149e Mon Sep 17 00:00:00 2001 From: Christopher Koch Date: Sat, 29 Dec 2018 09:16:15 -0800 Subject: dhcpv4: simplify marshaling options to binary. - Consolidate writing the option code and length to Options.Marshal rather than doing it in each individual option. - Use uio in marshaling code. --- dhcpv4/option_parameter_request_list.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'dhcpv4/option_parameter_request_list.go') diff --git a/dhcpv4/option_parameter_request_list.go b/dhcpv4/option_parameter_request_list.go index 8516e3b..e3eb7ff 100644 --- a/dhcpv4/option_parameter_request_list.go +++ b/dhcpv4/option_parameter_request_list.go @@ -33,11 +33,11 @@ func (o *OptParameterRequestList) Code() OptionCode { // ToBytes returns a serialized stream of bytes for this option. func (o *OptParameterRequestList) ToBytes() []byte { - ret := []byte{byte(o.Code()), byte(o.Length())} + buf := uio.NewBigEndianBuffer(nil) for _, req := range o.RequestedOpts { - ret = append(ret, byte(req)) + buf.Write8(uint8(req)) } - return ret + return buf.Data() } // String returns a human-readable string for this option. -- cgit v1.2.3