Rooba
/
perkeep
mirror of https://github.com/perkeep/perkeep.git
1
1
Fork 0
Code Issues Packages Projects Releases Wiki Activity

schema: document PopulateParts, add a missing error check 

Change-Id: Ic9d65fe3e56c4233f481254c6f730a8b382fb4ba
This commit is contained in:
Brad Fitzpatrick 2012-10-29 00:10:31 +01:00
parent 0ddc9dcadb
commit 044194b746
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
pkg/schema/schema.go
View File

@ -520,6 +520,10 @@ func NewCommonFileMap(fileName string, fi os.FileInfo) Map {
return m
}
// PopulateParts populates the "parts" field of m with the provided
// parts. The sum of the sizes of parts must match the provided size
// or an error is returned. Also, each BytesPart may only contain either
// a BytesPart or a BlobRef, but not both.
func PopulateParts(m Map, size int64, parts []BytesPart) error {
sumSize := int64(0)
mparts := make([]Map, len(parts))
@ -528,11 +532,13 @@ func PopulateParts(m Map, size int64, parts []BytesPart) error {
mparts[idx] = mpart
switch {
case part.BlobRef != nil && part.BytesRef != nil:
return errors.New("schema: part contains both blobRef and bytesRef")
return errors.New("schema: part contains both BlobRef and BytesRef")
case part.BlobRef != nil:
mpart["blobRef"] = part.BlobRef.String()
case part.BytesRef != nil:
mpart["bytesRef"] = part.BytesRef.String()
default:
return errors.New("schema: part must contain either a BlobRef or BytesRef")
}
mpart["size"] = part.Size
sumSize += int64(part.Size)