Merge 6b33958 into 3cbc1ae

osc/osc.go

@@ -1014,7 +1014,7 @@ func writePaddedString(str string, buf *bytes.Buffer) (int, error) {
 // padBytesNeeded determines how many bytes are needed to fill up to the next 4
 // byte length.
 func padBytesNeeded(elementLen int) int {
-	return 4*(elementLen/4+1) - elementLen
+	return -elementLen%4
 }
 
 ////

osc/osc_test.go

@@ -356,7 +356,7 @@ func TestReadPaddedString(t *testing.T) {
 		s   string // resulting string
 	}{
 		{[]byte{'t', 'e', 's', 't', 's', 't', 'r', 'i', 'n', 'g', 0, 0}, 12, "teststring"},
-		{[]byte{'t', 'e', 's', 't', 0, 0, 0, 0}, 8, "test"},
+		{[]byte{'t', 'e', 's', 't'}, 4, "test"},
 	} {
 		buf := bytes.NewBuffer(tt.buf)
 		s, n, err := readPaddedString(bufio.NewReader(buf))
@@ -391,28 +391,33 @@ func TestWritePaddedString(t *testing.T) {
 func TestPadBytesNeeded(t *testing.T) {
 	var n int
 	n = padBytesNeeded(4)
-	if n != 4 {
-		t.Errorf("Number of pad bytes should be 4 and is: %d", n)
+	if n != 0 {
+		t.Errorf("Number of pad bytes should be 0 and is: %d", n)
 	}
 
 	n = padBytesNeeded(3)
 	if n != 1 {
 		t.Errorf("Number of pad bytes should be 1 and is: %d", n)
 	}
 
+	n = padBytesNeeded(2)
+	if n != 2 {
+		t.Errorf("Number of pad bytes should be 2 and is: %d", n)
+	}
+
 	n = padBytesNeeded(1)
 	if n != 3 {
 		t.Errorf("Number of pad bytes should be 3 and is: %d", n)
 	}
 
 	n = padBytesNeeded(0)
-	if n != 4 {
-		t.Errorf("Number of pad bytes should be 4 and is: %d", n)
+	if n != 0 {
+		t.Errorf("Number of pad bytes should be 0 and is: %d", n)
 	}
 
 	n = padBytesNeeded(32)
-	if n != 4 {
-		t.Errorf("Number of pad bytes should be 4 and is: %d", n)
+	if n != 0 {
+		t.Errorf("Number of pad bytes should be 0 and is: %d", n)
 	}
 
 	n = padBytesNeeded(63)