Remove redundant conditions in unmarshalSlice()

fi.size is checked 3 times, one time is enough.
abema · Sep 30, 2023 · fb51aa1 · fb51aa1
1 parent 1261cd9
commit fb51aa1
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/marshaller.go b/marshaller.go
@@ -422,7 +422,7 @@ func (u *unmarshaller) unmarshalSlice(v reflect.Value, fi *fieldInstance) error
 		return fmt.Errorf("out of memory: requestedSize=%d", length)
 	}
 
-	if fi.size != 0 && fi.size%8 == 0 && u.rbits%8 == 0 && elemType.Kind() == reflect.Uint8 && fi.size == 8 {
+	if u.rbits%8 == 0 && elemType.Kind() == reflect.Uint8 && fi.size == 8 {
 		totalSize := length * uint64(fi.size) / 8
 		capacity := totalSize
 		if u.dst.GetType() != BoxTypeMdat() && capacity > maxInitialSliceCapacity {