nsscash: improve comments

[nsscash/nsscash.git] / group.go

diff --git a/group.go b/group.go
index b7f7d1b8e8b9c4ce71252570c5d9cc6182b972d9..0ad86711431d068ff0922ceace8c39cdba01a1a0 100644 (file)
--- a/group.go
+++ b/group.go
@@ -45,7 +45,7 @@ type GroupKey struct {
        Name    string
        Passwd  string
        Gid     uint64
-       Members string
+       Members string // "," separated
 }
 
 func toKey(g Group) GroupKey {
@@ -58,7 +58,7 @@ func toKey(g Group) GroupKey {
 }
 
 // ParseGroups parses a file in the format of /etc/group and returns all
-// entries as Group structs.
+// entries as slice of Group structs.
 func ParseGroups(r io.Reader) ([]Group, error) {
        var res []Group
 
@@ -114,10 +114,7 @@ func SerializeGroup(g Group) []byte {
        offPasswd := uint16(data.Len())
        data.Write([]byte(g.Passwd))
        data.WriteByte(0)
-       // Padding to align the following uint16
-       if data.Len()%2 != 0 {
-               data.WriteByte(0)
-       }
+       alignBufferTo(&data, 2) // align the following uint16
        offMemOff := uint16(data.Len())
        // Offsets for group members
        offMem := offMemOff + 2*uint16(len(mems_off))
@@ -154,12 +151,7 @@ func SerializeGroup(g Group) []byte {
        res.Write(data.Bytes())
        // We must pad each entry so that all uint64 at the beginning of the
        // struct are 8 byte aligned
-       l := res.Len()
-       if l%8 != 0 {
-               for i := 0; i < 8-l%8; i++ {
-                       res.WriteByte(0)
-               }
-       }
+       alignBufferTo(&res, 8)
 
        return res.Bytes()
 }