Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

c++ remove conversion warning in MapEntryFuncs::ByteSizeLong #6766

Merged
merged 1 commit into from Oct 18, 2019
Merged

c++ remove conversion warning in MapEntryFuncs::ByteSizeLong #6766

merged 1 commit into from Oct 18, 2019

Conversation

chrisse74
Copy link
Contributor

Removes following conversion warning:

google/protobuf/map_entry_lite.h(131): warning C4267: 'argument': conversion from 'size_t' to 'google::protobuf::uint32', possible loss of data
google/protobuf/map_entry_lite.h(127): note: while compiling class template member function 'size_t google::protobuf::internal::MapEntryFuncs<Key,Value,google::protobuf::internal::WireFormatLite::TYPE_STRING,google::protobuf::internal::WireFormatLite::TYPE_BYTES>::ByteSizeLong(const Key &,const Value &)'
        with
       [
            Key=std::string,
            Value=std::string
        ]
note: see reference to function template instantiation 'size_t google::protobuf::internal::MapEntryFuncs<Key,Value,google::protobuf::internal::WireFormatLite::TYPE_STRING,google::protobuf::internal::WireFormatLite::TYPE_BYTES>::ByteSizeLong(const Key &,const Value &)' being compiled
        with
        [
            Key=std::string,
            Value=std::string
        ]
note: see reference to class template instantiation 'google::protobuf::internal::MapEntryFuncs<Key,Value,google::protobuf::internal::WireFormatLite::TYPE_STRING,google::protobuf::internal::WireFormatLite::TYPE_BYTES>' being compiled
        with
        [
            Key=std::string,
            Value=std::string
        ]

@acozzette acozzette self-assigned this Oct 15, 2019
@acozzette acozzette merged commit 6f0f382 into protocolbuffers:master Oct 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants