Skip to content

Commit f17fe7d

Browse files
Automated Code Change
PiperOrigin-RevId: 814956417
1 parent 959ed79 commit f17fe7d

File tree

3 files changed

+11
-7
lines changed

3 files changed

+11
-7
lines changed

src/google/protobuf/descriptor.cc

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9796,7 +9796,8 @@ bool DescriptorBuilder::OptionInterpreter::InterpretSingleOption(
97969796

97979797
// First set the value on the UnknownFieldSet corresponding to the
97989798
// innermost message.
9799-
std::unique_ptr<UnknownFieldSet> unknown_fields(new UnknownFieldSet());
9799+
std::unique_ptr<UnknownFieldSet> unknown_fields =
9800+
std::make_unique<UnknownFieldSet>();
98009801
if (!SetOptionValue(field, unknown_fields.get())) {
98019802
return false; // SetOptionValue() already added the error.
98029803
}
@@ -9806,8 +9807,8 @@ bool DescriptorBuilder::OptionInterpreter::InterpretSingleOption(
98069807
for (std::vector<const FieldDescriptor*>::reverse_iterator iter =
98079808
intermediate_fields.rbegin();
98089809
iter != intermediate_fields.rend(); ++iter) {
9809-
std::unique_ptr<UnknownFieldSet> parent_unknown_fields(
9810-
new UnknownFieldSet());
9810+
std::unique_ptr<UnknownFieldSet> parent_unknown_fields =
9811+
std::make_unique<UnknownFieldSet>();
98119812
switch ((*iter)->type()) {
98129813
case FieldDescriptor::TYPE_MESSAGE: {
98139814
std::string outstr;

src/google/protobuf/extension_set_unittest.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -543,7 +543,7 @@ TEST(ExtensionSetTest, ArenaSetAllocatedMessageAndRelease) {
543543

544544
TEST(ExtensionSetTest, SwapExtensionBothFullWithArena) {
545545
Arena arena1;
546-
std::unique_ptr<Arena> arena2(new Arena());
546+
std::unique_ptr<Arena> arena2 = std::make_unique<Arena>();
547547

548548
unittest::TestAllExtensions* message1 =
549549
Arena::Create<unittest::TestAllExtensions>(&arena1);

src/google/protobuf/text_format_unittest.cc

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1797,7 +1797,8 @@ class TextFormatParserTest : public testing::Test {
17971797
protected:
17981798
void ExpectFailure(const std::string& input, const std::string& message,
17991799
int line, int col) {
1800-
std::unique_ptr<unittest::TestAllTypes> proto(new unittest::TestAllTypes);
1800+
std::unique_ptr<unittest::TestAllTypes> proto =
1801+
std::make_unique<unittest::TestAllTypes>();
18011802
ExpectFailure(input, message, line, col, proto.get());
18021803
}
18031804

@@ -1867,7 +1868,8 @@ class TextFormatParserTest : public testing::Test {
18671868
};
18681869

18691870
TEST_F(TextFormatParserTest, ParseInfoTreeBuilding) {
1870-
std::unique_ptr<unittest::TestAllTypes> message(new unittest::TestAllTypes);
1871+
std::unique_ptr<unittest::TestAllTypes> message =
1872+
std::make_unique<unittest::TestAllTypes>();
18711873
const Descriptor* d = message->GetDescriptor();
18721874

18731875
std::string stringData =
@@ -1934,7 +1936,8 @@ TEST_F(TextFormatParserTest, ParseInfoTreeBuilding) {
19341936
}
19351937

19361938
TEST_F(TextFormatParserTest, ParseFieldValueFromString) {
1937-
std::unique_ptr<unittest::TestAllTypes> message(new unittest::TestAllTypes);
1939+
std::unique_ptr<unittest::TestAllTypes> message =
1940+
std::make_unique<unittest::TestAllTypes>();
19381941
const Descriptor* d = message->GetDescriptor();
19391942

19401943
#define EXPECT_FIELD(name, value, valuestring) \

0 commit comments

Comments
 (0)