all: more typos

Change-Id: I7be32dd41cc3fdf2202d1dcb939790f2afba6505

Author: NathanBaulch

compiler/protogen/protogen_opaque.go

@@ -21,7 +21,7 @@ func opaqueNewMessageHook(message *Message) {
 	// New name mangling scheme: Add a '_' between method base
 	// name (Get, Set, Clear etc) and original field name if
 	// needed.  As a special case, there is one globally reserved
-	// name, e.g. "Build" thet still results in actual renaming of
+	// name, e.g. "Build" that still results in actual renaming of
 	// the builder field like in the old scheme.  We begin by
 	// taking care of this special case.
 	for _, field := range message.Fields {
@@ -45,7 +45,7 @@ func opaqueNewMessageHook(message *Message) {
 		camelCases[field.camelCase] = true
 
 	}
-	// For each field, check if any of it's methods would clash with an original field name
+	// For each field, check if any of its methods would clash with an original field name
 	for _, field := range message.Fields {
 		// Every field (except the union fields, that are taken care of separately) has
 		// a Get and a Set method.

internal/cmd/generate-types/proto.go

@@ -104,7 +104,7 @@ func (g GoType) NullableStar() Expr {
 	return "*"
 }
 
-// OpaqueNullablePointerMethod is the "internal/impl".pointer method used to access a opaque nullable pointer to this type.
+// OpaqueNullablePointerMethod is the "internal/impl".pointer method used to access an opaque nullable pointer to this type.
 func (g GoType) OpaqueNullablePointerMethod() Expr {
 	switch g {
 	case GoString:
@@ -117,7 +117,7 @@ func (g GoType) OpaqueNullablePointerMethod() Expr {
 
 }
 
-// OpaqueNullableStar is the prefix for dereferencing a opaque nullable value of this type.
+// OpaqueNullableStar is the prefix for dereferencing an opaque nullable value of this type.
 func (g GoType) OpaqueNullableStar() Expr {
 	if g == GoString {
 		return "*" // Strings have indirection even in opaque

internal/impl/api_export_opaque.go

@@ -23,7 +23,7 @@ func (Export) UnmarshalField(msg any, fieldNum int32) {
 // &m.XXX_presence[1], num = 70)
 func (Export) Present(part *uint32, num uint32) bool {
 	// This hook will read an unprotected shadow presence set if
-	// we're unning under the race detector
+	// we're running under the race detector
 	raceDetectHookPresent(part, num)
 	return atomic.LoadUint32(part)&(1<<(num%32)) > 0
 }

internal/impl/lazy_buffersharing_test.go

@@ -140,7 +140,7 @@ func testNoExcessiveCopy(t *testing.T, unmarshalOpts proto.UnmarshalOptions) {
 	}
 	b = (*m3.XXX_lazyUnmarshalInfo).Protobuf
 	if m2len != 0 && len(b) == 0 {
-		t.Errorf("The lazy backing store for submessage is empty when it is not for the surronding message: %v.", m2len)
+		t.Errorf("The lazy backing store for submessage is empty when it is not for the surrounding message: %v.", m2len)
 	}
 	for i, x := range b {
 		if x != byte(0xFF) {

internal/impl/presence.go

@@ -57,7 +57,7 @@ func (p presence) ClearPresent(num uint32) {
 // LoadPresenceCache (together with PresentInCache) allows for a
 // cached version of checking for presence without re-reading the word
 // for every field. It is optimized for efficiency and assumes no
-// simltaneous mutation of the presence set (or at least does not have
+// simultaneous mutation of the presence set (or at least does not have
 // a problem with simultaneous mutation giving inconsistent results).
 func (p presence) LoadPresenceCache() (current uint32) {
 	if p.P == nil {
@@ -123,7 +123,7 @@ func atomicStoreShadowPresence(p **[]byte, v *[]byte) {
 
 // findPointerToRaceDetectData finds the preceding RaceDetectHookData
 // in a message by using pointer arithmetic. For the methods called
-// directy from generated code, we don't have a pointer to the
+// directly from generated code, we don't have a pointer to the
 // beginning of the presence set, but a pointer inside the array. As
 // we know the index of the bit we're manipulating (num), we can
 // calculate which element of the array ptr is pointing to. With that

internal/protolazy/bufferreader.go

@@ -258,7 +258,7 @@ func (b *BufferReader) SkipGroup(tag uint32) (err error) {
 	return nil
 }
 
-// skipVarint effiently skips a varint
+// skipVarint efficiently skips a varint
 func (b *BufferReader) SkipVarint() (err error) {
 	i := b.Pos
 

internal/protolazy/lazy.go

@@ -323,8 +323,8 @@ func (lazy *XXX_lazyUnmarshalInfo) FindFieldInProto(fieldNum uint32) (start, end
 // a repeated field), and bools indicating if field was found and if there
 // are multiple encodings of the field in the byte range.
 //
-// To hande the uncommon case where there are repeated encodings for the same
-// field which are not consecutive in the protobuf (so we need to returns
+// To handle the uncommon case where there are repeated encodings for the same
+// field which are not consecutive in the protobuf (so we need to return
 // multiple start/end offsets), we also return a slice multipleEntries.  If
 // multipleEntries is non-nil, then multiple entries were found, and the
 // values in the slice should be used, rather than start/end/found.

internal/race_test/lazy/lazy_race_test.go

@@ -184,7 +184,7 @@ func spawnCase(t *testing.T) {
 		t.Fatalf("Failed to find my own executable: %v", err)
 	}
 	c := exec.Command(ep, "--test.run="+t.Name())
-	// Set the environment variable so that we know we're in a subproceess when re-executed
+	// Set the environment variable so that we know we're in a subprocess when re-executed
 	c.Env = append(c.Env, envVar+"=true")
 	out, err := c.CombinedOutput()
 	// If we do not get an error, we fail in the parent process, otherwise we're good
@@ -310,9 +310,9 @@ func expandedLazy(m *testopaquepb.TestRequiredLazy) bool {
 	return rf.Pointer() != 0
 }
 
-// This test verifies all assumptions of TestParallellMarshalWithRequired
+// This test verifies all assumptions of TestParallelMarshalWithRequired
 // are (still) valid, to prevent the test from becoming a no-op (again).
-func TestParallellMarshalWithRequiredAssumptions(t *testing.T) {
+func TestParallelMarshalWithRequiredAssumptions(t *testing.T) {
 	b, err := proto.Marshal(fillRequiredLazy())
 	if err != nil {
 		t.Fatal(err)
@@ -347,12 +347,12 @@ func TestParallellMarshalWithRequiredAssumptions(t *testing.T) {
 	}
 }
 
-// TestParallellMarshalWithRequired runs two goroutines that marshal the same
+// TestParallelMarshalWithRequired runs two goroutines that marshal the same
 // message. Marshaling a message can result in lazily decoding said message,
 // provided the message contains any required fields. This test ensures that
 // said lazy decoding can happen without causing races in the other goroutine
 // that marshals the same message.
-func TestParallellMarshalWithRequired(t *testing.T) {
+func TestParallelMarshalWithRequired(t *testing.T) {
 	m := fillRequiredLazy()
 	b, err := proto.MarshalOptions{}.Marshal(m)
 	if err != nil {
@@ -432,7 +432,7 @@ func fillMixedOpaqueLazy() *mixedpb.OpaqueLazy {
 	}.Build()
 }
 
-func TestParallellMarshalMixed(t *testing.T) {
+func TestParallelMarshalMixed(t *testing.T) {
 	m := fillMixedOpaqueLazy()
 	b, err := proto.Marshal(m)
 	if err != nil {

reflect/protorange/range.go

@@ -71,7 +71,7 @@ type Options struct {
 //     Each field is preceded by a push of a [protopath.FieldAccess] step,
 //     followed by recursive application of the rules on the field value,
 //     and succeeded by a pop of that step.
-//     If the message has unknown fields, then push an [protopath.UnknownAccess] step
+//     If the message has unknown fields, then push a [protopath.UnknownAccess] step
 //     followed immediately by pop of that step.
 //
 //   - As an exception to the above rule, if the current message is a

reflect/protoreflect/type.go

@@ -214,7 +214,7 @@ type MessageDescriptor interface {
 	ExtensionRanges() FieldRanges
 	// ExtensionRangeOptions returns the ith extension range options.
 	//
-	// To avoid a dependency cycle, this method returns a proto.Message] value,
+	// To avoid a dependency cycle, this method returns a proto.Message value,
 	// which always contains a google.protobuf.ExtensionRangeOptions message.
 	// This method returns a typed nil-pointer if no options are present.
 	// The caller must import the descriptorpb package to use this.