Skip to content

Commit a561125

Browse files
committed
update function name, doc file, and spelling.
1 parent 6d0128c commit a561125

File tree

3 files changed

+51
-55
lines changed

3 files changed

+51
-55
lines changed

bson/bsoncodec/struct_codec.go

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -189,17 +189,17 @@ func (sc *StructCodec) EncodeValue(ec EncodeContext, vw bsonrw.ValueWriter, val
189189

190190
encoder := desc.encoder
191191

192-
var zero bool
192+
var empty bool
193193
if cz, ok := encoder.(CodecZeroer); ok {
194-
zero = cz.IsTypeZero(rv.Interface())
194+
empty = cz.IsTypeZero(rv.Interface())
195195
} else if rv.Kind() == reflect.Interface {
196-
// isZero will not treat an interface rv as an interface, so we need to check for the
197-
// zero interface separately.
198-
zero = rv.IsNil()
196+
// isEmpty will not treat an interface rv as an interface, so we need to check for the
197+
// nil interface separately.
198+
empty = rv.IsNil()
199199
} else {
200-
zero = isZero(rv, sc.EncodeOmitDefaultStruct || ec.omitZeroStruct)
200+
empty = isEmpty(rv, sc.EncodeOmitDefaultStruct || ec.omitZeroStruct)
201201
}
202-
if desc.omitEmpty && zero {
202+
if desc.omitEmpty && empty {
203203
continue
204204
}
205205

@@ -391,7 +391,7 @@ func (sc *StructCodec) DecodeValue(dc DecodeContext, vr bsonrw.ValueReader, val
391391
return nil
392392
}
393393

394-
func isZero(v reflect.Value, omitZeroStruct bool) bool {
394+
func isEmpty(v reflect.Value, omitZeroStruct bool) bool {
395395
kind := v.Kind()
396396
if (kind != reflect.Ptr || !v.IsNil()) && v.Type().Implements(tZeroer) {
397397
return v.Interface().(Zeroer).IsZero()
@@ -413,7 +413,7 @@ func isZero(v reflect.Value, omitZeroStruct bool) bool {
413413
if ff.PkgPath != "" && !ff.Anonymous {
414414
continue // Private field
415415
}
416-
if !isZero(v.Field(i), omitZeroStruct) {
416+
if !isEmpty(v.Field(i), omitZeroStruct) {
417417
return false
418418
}
419419
}

bson/bsoncodec/struct_codec_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -163,8 +163,8 @@ func TestIsZero(t *testing.T) {
163163
t.Run(tc.description, func(t *testing.T) {
164164
t.Parallel()
165165

166-
got := isZero(reflect.ValueOf(tc.value), tc.omitZeroStruct)
167-
assert.Equal(t, tc.want, got, "expected and actual isZero return are different")
166+
got := isEmpty(reflect.ValueOf(tc.value), tc.omitZeroStruct)
167+
assert.Equal(t, tc.want, got, "expected and actual isEmpty return are different")
168168
})
169169
}
170170
}

bson/doc.go

Lines changed: 40 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@
66

77
// Package bson is a library for reading, writing, and manipulating BSON. BSON is a binary serialization format used to
88
// store documents and make remote procedure calls in MongoDB. The BSON specification is located at https://bsonspec.org.
9-
// The BSON library handles marshalling and unmarshalling of values through a configurable codec system. For a description
10-
// of the codec system and examples of registering custom codecs, see the bsoncodec package. For additional information and
11-
// usage examples, check out the [Work with BSON] page in the Go Driver docs site.
9+
// The BSON library handles marshaling and unmarshaling of values through a configurable codec system. For a description
10+
// of the codec system and examples of registering custom codecs, see the bsoncodec package. For additional information
11+
// and usage examples, check out the [Work with BSON] page in the Go Driver docs site.
1212
//
1313
// # Raw BSON
1414
//
@@ -38,7 +38,7 @@
3838
// bson.D{{"foo", "bar"}, {"hello", "world"}, {"pi", 3.14159}}
3939
// bson.M{"foo": "bar", "hello": "world", "pi": 3.14159}
4040
//
41-
// When decoding BSON to a D or M, the following type mappings apply when unmarshalling:
41+
// When decoding BSON to a D or M, the following type mappings apply when unmarshaling:
4242
//
4343
// 1. BSON int32 unmarshals to an int32.
4444
// 2. BSON int64 unmarshals to an int64.
@@ -62,7 +62,7 @@
6262
// 20. BSON DBPointer unmarshals to a primitive.DBPointer.
6363
// 21. BSON symbol unmarshals to a primitive.Symbol.
6464
//
65-
// The above mappings also apply when marshalling a D or M to BSON. Some other useful marshalling mappings are:
65+
// The above mappings also apply when marshaling a D or M to BSON. Some other useful marshaling mappings are:
6666
//
6767
// 1. time.Time marshals to a BSON datetime.
6868
// 2. int8, int16, and int32 marshal to a BSON int32.
@@ -72,73 +72,69 @@
7272
// 5. uint8 and uint16 marshal to a BSON int32.
7373
// 6. uint, uint32, and uint64 marshal to a BSON int32 if the value is between math.MinInt32 and math.MaxInt32,
7474
// inclusive, and BSON int64 otherwise.
75-
// 7. BSON null and undefined values will unmarshal into the zero value of a field (e.g. unmarshalling a BSON null or
75+
// 7. BSON null and undefined values will unmarshal into the zero value of a field (e.g. unmarshaling a BSON null or
7676
// undefined value into a string will yield the empty string.).
7777
//
7878
// # Structs
7979
//
80-
// Structs can be marshalled/unmarshalled to/from BSON or Extended JSON. When transforming structs to/from BSON or Extended
80+
// Structs can be marshaled/unmarshaled to/from BSON or Extended JSON. When transforming structs to/from BSON or Extended
8181
// JSON, the following rules apply:
8282
//
83-
// 1. Only exported fields in structs will be marshalled or unmarshalled.
83+
// 1. Only exported fields in structs will be marshaled or unmarshaled.
8484
//
85-
// 2. When marshalling a struct, each field will be lowercased to generate the key for the corresponding BSON element.
85+
// 2. When marshaling a struct, each field will be lowercased to generate the key for the corresponding BSON element.
8686
// For example, a struct field named "Foo" will generate key "foo". This can be overridden via a struct tag (e.g.
8787
// `bson:"fooField"` to generate key "fooField" instead).
8888
//
89-
// 3. An embedded struct field is marshalled as a subdocument. The key will be the lowercased name of the field's type.
89+
// 3. An embedded struct field is marshaled as a subdocument. The key will be the lowercased name of the field's type.
9090
//
91-
// 4. A pointer field is marshalled as the underlying type if the pointer is non-nil. If the pointer is nil, it is
92-
// marshalled as a BSON null value.
91+
// 4. A pointer field is marshaled as the underlying type if the pointer is non-nil. If the pointer is nil, it is
92+
// marshaled as a BSON null value.
9393
//
94-
// 5. When unmarshalling, a field of type interface{} will follow the D/M type mappings listed above. BSON documents
95-
// unmarshalled into an interface{} field will be unmarshalled as a D.
94+
// 5. When unmarshaling, a field of type interface{} will follow the D/M type mappings listed above. BSON documents
95+
// unmarshaled into an interface{} field will be unmarshaled as a D.
9696
//
9797
// The encoding of each struct field can be customized by the "bson" struct tag.
9898
//
9999
// This tag behavior is configurable, and different struct tag behavior can be configured by initializing a new
100-
// bsoncodec.StructCodec with the desired tag parser and registering that StructCodec onto the Registry. By default, JSON tags
101-
// are not honored, but that can be enabled by creating a StructCodec with JSONFallbackStructTagParser, like below:
100+
// bsoncodec.StructCodec with the desired tag parser and registering that StructCodec onto the Registry. By default, JSON
101+
// tags are not honored, but that can be enabled by creating a StructCodec with JSONFallbackStructTagParser, like below:
102102
//
103103
// Example:
104104
//
105105
// structcodec, _ := bsoncodec.NewStructCodec(bsoncodec.JSONFallbackStructTagParser)
106106
//
107107
// The bson tag gives the name of the field, possibly followed by a comma-separated list of options.
108-
// The name may be empty in order to specify options without overriding the default field name. The following options can be used
109-
// to configure behavior:
110-
//
111-
// 1. omitempty: If the omitempty struct tag is specified on a field, the field will not be marshalled if it is set to
112-
// the zero value. Fields with language primitive types such as integers, booleans, and strings are considered empty if
113-
// their value is equal to the zero value for the type (i.e. 0 for integers, false for booleans, and "" for strings).
114-
// Slices, maps, and arrays are considered empty if they are of length zero. Interfaces and pointers are considered
115-
// empty if their value is nil. By default, structs are only considered empty if the struct type implements the
116-
// bsoncodec.Zeroer interface and the IsZero method returns true. Struct fields whose types do not implement Zeroer are
117-
// never considered empty and will be marshalled as embedded documents.
108+
// The name may be empty in order to specify options without overriding the default field name. The following options can
109+
// be used to configure behavior:
110+
//
111+
// 1. omitempty: If the omitempty struct tag is specified on a field, the field will be omitted from the marshaling if
112+
// the field has an empty value, defined as false, 0, a nil pointer, a nil interface value, and any empty array,
113+
// slice, map, or string.
118114
// NOTE: It is recommended that this tag be used for all slice and map fields.
119115
//
120116
// 2. minsize: If the minsize struct tag is specified on a field of type int64, uint, uint32, or uint64 and the value of
121-
// the field can fit in a signed int32, the field will be serialized as a BSON int32 rather than a BSON int64. For other
122-
// types, this tag is ignored.
117+
// the field can fit in a signed int32, the field will be serialized as a BSON int32 rather than a BSON int64. For
118+
// other types, this tag is ignored.
123119
//
124-
// 3. truncate: If the truncate struct tag is specified on a field with a non-float numeric type, BSON doubles unmarshalled
125-
// into that field will be truncated at the decimal point. For example, if 3.14 is unmarshalled into a field of type int,
126-
// it will be unmarshalled as 3. If this tag is not specified, the decoder will throw an error if the value cannot be
127-
// decoded without losing precision. For float64 or non-numeric types, this tag is ignored.
120+
// 3. truncate: If the truncate struct tag is specified on a field with a non-float numeric type, BSON doubles
121+
// unmarshaled into that field will be truncated at the decimal point. For example, if 3.14 is unmarshaled into a
122+
// field of type int, it will be unmarshaled as 3. If this tag is not specified, the decoder will throw an error if
123+
// the value cannot be decoded without losing precision. For float64 or non-numeric types, this tag is ignored.
128124
//
129125
// 4. inline: If the inline struct tag is specified for a struct or map field, the field will be "flattened" when
130-
// marshalling and "un-flattened" when unmarshalling. This means that all of the fields in that struct/map will be
131-
// pulled up one level and will become top-level fields rather than being fields in a nested document. For example, if a
132-
// map field named "Map" with value map[string]interface{}{"foo": "bar"} is inlined, the resulting document will be
133-
// {"foo": "bar"} instead of {"map": {"foo": "bar"}}. There can only be one inlined map field in a struct. If there are
134-
// duplicated fields in the resulting document when an inlined struct is marshalled, the inlined field will be overwritten.
135-
// If there are duplicated fields in the resulting document when an inlined map is marshalled, an error will be returned.
136-
// This tag can be used with fields that are pointers to structs. If an inlined pointer field is nil, it will not be
137-
// marshalled. For fields that are not maps or structs, this tag is ignored.
138-
//
139-
// # Marshalling and Unmarshalling
140-
//
141-
// Manually marshalling and unmarshalling can be done with the Marshal and Unmarshal family of functions.
126+
// marshaling and "un-flattened" when unmarshaling. This means that all of the fields in that struct/map will be
127+
// pulled up one level and will become top-level fields rather than being fields in a nested document. For example,
128+
// if a map field named "Map" with value map[string]interface{}{"foo": "bar"} is inlined, the resulting document will
129+
// be {"foo": "bar"} instead of {"map": {"foo": "bar"}}. There can only be one inlined map field in a struct. If
130+
// there are duplicated fields in the resulting document when an inlined struct is marshaled, the inlined field will
131+
// be overwritten. If there are duplicated fields in the resulting document when an inlined map is marshaled, an
132+
// error will be returned. This tag can be used with fields that are pointers to structs. If an inlined pointer field
133+
// is nil, it will not be marshaled. For fields that are not maps or structs, this tag is ignored.
134+
//
135+
// # Marshaling and Unmarshaling
136+
//
137+
// Manually marshaling and unmarshaling can be done with the Marshal and Unmarshal family of functions.
142138
//
143139
// [Work with BSON]: https://www.mongodb.com/docs/drivers/go/current/fundamentals/bson/
144140
package bson

0 commit comments

Comments
 (0)