From 9659ea9e56bfc34f67d7ce73f946d5b58183a833 Mon Sep 17 00:00:00 2001 From: Feng Xiao Date: Mon, 2 Nov 2015 12:39:27 -0800 Subject: Fix compilation errors when built internally. 1. mathlimits.h must be included before the inclusion of cmath (which gtest/gtest.h seems to include). 2. hash function for StringPiece doesn't work. Change-Id: I358a25d941a25b10b39fe76780eda41557699811 --- src/google/protobuf/util/field_comparator_test.cc | 3 ++- src/google/protobuf/util/internal/protostream_objectwriter.cc | 2 +- src/google/protobuf/util/internal/protostream_objectwriter.h | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/google/protobuf/util/field_comparator_test.cc b/src/google/protobuf/util/field_comparator_test.cc index 748c7d11..d3d34602 100644 --- a/src/google/protobuf/util/field_comparator_test.cc +++ b/src/google/protobuf/util/field_comparator_test.cc @@ -34,9 +34,10 @@ #include #include -#include #include +#include + namespace google { namespace protobuf { namespace util { diff --git a/src/google/protobuf/util/internal/protostream_objectwriter.cc b/src/google/protobuf/util/internal/protostream_objectwriter.cc index 08a2fb9a..0958997c 100644 --- a/src/google/protobuf/util/internal/protostream_objectwriter.cc +++ b/src/google/protobuf/util/internal/protostream_objectwriter.cc @@ -602,7 +602,7 @@ void ProtoStreamObjectWriter::ProtoElement::TakeOneofIndex(int32 index) { bool ProtoStreamObjectWriter::ProtoElement::InsertMapKeyIfNotPresent( StringPiece map_key) { - return InsertIfNotPresent(&map_keys_, map_key); + return InsertIfNotPresent(&map_keys_, map_key.ToString()); } inline void ProtoStreamObjectWriter::InvalidName(StringPiece unknown_name, diff --git a/src/google/protobuf/util/internal/protostream_objectwriter.h b/src/google/protobuf/util/internal/protostream_objectwriter.h index 19f747fe..6e133679 100644 --- a/src/google/protobuf/util/internal/protostream_objectwriter.h +++ b/src/google/protobuf/util/internal/protostream_objectwriter.h @@ -305,7 +305,7 @@ class LIBPROTOBUF_EXPORT ProtoStreamObjectWriter : public StructuredObjectWriter // Set of map keys already seen for the type_. Used to validate incoming // messages so no map key appears more than once. - hash_set map_keys_; + hash_set map_keys_; GOOGLE_DISALLOW_IMPLICIT_CONSTRUCTORS(ProtoElement); }; -- cgit v1.2.3