Skip to content
This repository was archived by the owner on Feb 23, 2026. It is now read-only.

Commit 3a96248

Browse files
Apply suggestions from code review
Co-authored-by: Chalmer Lowe <chalmerlowe@google.com>
1 parent 18e10e8 commit 3a96248

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

proto/message.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -930,10 +930,10 @@ def _message_to_map(
930930
"""
931931

932932
# The `including_default_value_fields` argument was removed from protobuf 5.x
933-
# and replaced with `always_print_fields_with_no_presence` which very similar but has
933+
# and replaced with `always_print_fields_with_no_presence` which is similar but
934934
# handles optional fields consistently by not affecting them.
935-
# The old flag accidentally had inconsistent behavior between proto2
936-
# optional and proto3 optional fields.
935+
# The old flag accidentally had inconsistent behavior between optional fields
936+
# in proto2 and proto3.
937937
print_fields = cls._normalize_print_fields_without_presence(
938938
always_print_fields_with_no_presence, including_default_value_fields
939939
)
@@ -953,7 +953,7 @@ def _message_to_map(
953953
"`float_precision` was removed in Protobuf 7.x+, and will be ignored."
954954
)
955955
warnings.warn(warning_msg, DeprecationWarning, stacklevel=3)
956-
# supress similar float_precision warning from protobuf library
956+
# suppress similar float_precision warning from protobuf library.
957957
with warnings.catch_warnings():
958958
warnings.simplefilter("ignore", category=UserWarning)
959959
return map_fn(cls.pb(instance), **kwargs)

0 commit comments

Comments
 (0)