diff options
| author | Shulhan <ms@kilabit.info> | 2021-10-14 21:30:49 +0700 |
|---|---|---|
| committer | Shulhan <ms@kilabit.info> | 2021-10-14 21:30:49 +0700 |
| commit | 2080593f8856863d8819062eede8ea59036b3299 (patch) | |
| tree | 1d5b302f754c014506c0bdb97f471d4217fcc26e /lib/bytes/bytes.go | |
| parent | 8a0352321439e1fe10cfeeebf94372af9c9f7c04 (diff) | |
| download | pakakeh.go-2080593f8856863d8819062eede8ea59036b3299.tar.xz | |
lib/bytes: refactoring AppendXxx functions
Previously, we pass pointer to slice on AppendInt16, AppendInt32,
AppendUint16, and AppendUint32 functions. This model of function
signature is not a Go idiom. It is written when I am still new to
Go.
Diffstat (limited to 'lib/bytes/bytes.go')
| -rw-r--r-- | lib/bytes/bytes.go | 44 |
1 files changed, 24 insertions, 20 deletions
diff --git a/lib/bytes/bytes.go b/lib/bytes/bytes.go index 6c1e6db8..f5a60f3c 100644 --- a/lib/bytes/bytes.go +++ b/lib/bytes/bytes.go @@ -13,39 +13,43 @@ import ( ) // -// AppendInt16 into slice of byte. +// AppendInt16 append an int16 value into slice of byte. // -func AppendInt16(data *[]byte, v int16) { - *data = append(*data, byte(v>>8)) - *data = append(*data, byte(v)) +func AppendInt16(data []byte, v int16) []byte { + data = append(data, byte(v>>8)) + data = append(data, byte(v)) + return data } // -// AppendInt32 into slice of byte. +// AppendInt32 append an int32 value into slice of byte. // -func AppendInt32(data *[]byte, v int32) { - *data = append(*data, byte(v>>24)) - *data = append(*data, byte(v>>16)) - *data = append(*data, byte(v>>8)) - *data = append(*data, byte(v)) +func AppendInt32(data []byte, v int32) []byte { + data = append(data, byte(v>>24)) + data = append(data, byte(v>>16)) + data = append(data, byte(v>>8)) + data = append(data, byte(v)) + return data } // -// AppendUint16 into slice of byte. +// AppendUint16 append an uint16 value into slice of byte. // -func AppendUint16(data *[]byte, v uint16) { - *data = append(*data, byte(v>>8)) - *data = append(*data, byte(v)) +func AppendUint16(data []byte, v uint16) []byte { + data = append(data, byte(v>>8)) + data = append(data, byte(v)) + return data } // -// AppendUint32 into slice of byte. +// AppendUint32 append an uint32 value into slice of byte. // -func AppendUint32(data *[]byte, v uint32) { - *data = append(*data, byte(v>>24)) - *data = append(*data, byte(v>>16)) - *data = append(*data, byte(v>>8)) - *data = append(*data, byte(v)) +func AppendUint32(data []byte, v uint32) []byte { + data = append(data, byte(v>>24)) + data = append(data, byte(v>>16)) + data = append(data, byte(v>>8)) + data = append(data, byte(v)) + return data } // |
