return uint64 for parseSize

Signed-off-by: allencloud <allen.sun@daocloud.io>

Author: allencloud

size.go

@@ -63,34 +63,34 @@ func BytesSize(size float64) string {
 
 // FromHumanSize returns an integer from a human-readable specification of a
 // size using SI standard (eg. "44kB", "17MB").
-func FromHumanSize(size string) (int64, error) {
+func FromHumanSize(size string) (uint64, error) {
 	return parseSize(size, decimalMap)
 }
 
 // RAMInBytes parses a human-readable string representing an amount of RAM
 // in bytes, kibibytes, mebibytes, gibibytes, or tebibytes and
-// returns the number of bytes, or -1 if the string is unparseable.
+// returns the number of bytes.
 // Units are case-insensitive, and the 'b' suffix is optional.
-func RAMInBytes(size string) (int64, error) {
+func RAMInBytes(size string) (uint64, error) {
 	return parseSize(size, binaryMap)
 }
 
 // Parses the human-readable size string into the amount it represents.
-func parseSize(sizeStr string, uMap unitMap) (int64, error) {
+func parseSize(sizeStr string, uMap unitMap) (uint64, error) {
 	matches := sizeRegex.FindStringSubmatch(sizeStr)
 	if len(matches) != 4 {
-		return -1, fmt.Errorf("invalid size: '%s'", sizeStr)
+		return 0, fmt.Errorf("invalid size: '%s'", sizeStr)
 	}
 
 	size, err := strconv.ParseFloat(matches[1], 64)
 	if err != nil {
-		return -1, err
+		return 0, err
 	}
 
 	unitPrefix := strings.ToLower(matches[3])
 	if mul, ok := uMap[unitPrefix]; ok {
 		size *= float64(mul)
 	}
 
-	return int64(size), nil
+	return uint64(size), nil
 }

size_test.go

@@ -142,15 +142,15 @@ func assertEquals(t *testing.T, expected, actual interface{}) {
 }
 
 // func that maps to the parse function signatures as testing abstraction
-type parseFn func(string) (int64, error)
+type parseFn func(string) (uint64, error)
 
 // Define 'String()' for pretty-print
 func (fn parseFn) String() string {
 	fnName := runtime.FuncForPC(reflect.ValueOf(fn).Pointer()).Name()
 	return fnName[strings.LastIndex(fnName, ".")+1:]
 }
 
-func assertSuccessEquals(t *testing.T, expected int64, fn parseFn, arg string) {
+func assertSuccessEquals(t *testing.T, expected uint64, fn parseFn, arg string) {
 	res, err := fn(arg)
 	if err != nil || res != expected {
 		t.Errorf("%s(\"%s\") -> expected '%d' but got '%d' with error '%v'", fn, arg, expected, res, err)