commit 153f87a2fd2972f862a88976bb884a90a1f9fd7d
author Steven Moreland <smoreland@google.com> Tue Feb 28 09:42:26 2017 -0800
committer Steven Moreland <smoreland@google.com> Tue Feb 28 09:49:27 2017 -0800
tree c9b47be56f7b3ea9fb97fc7c39efc145d24543a5
parent 311f9dd097a5ffb5d25b245c7442b1270a21df07

Make operator= and constructor behavior consistent.

hidl_string(nullptr) == "". Now so does operator=(nullptr).

Bug: 35421240
Test: libhidl_test
Change-Id: Ib62fa71ab47a407b7017c809783dd5f421b98e39

test_main.cpp

diff --git a/test_main.cpp b/test_main.cpp
index 6bd7f83..4f59371 100644
--- a/test_main.cpp
+++ b/test_main.cpp
@@ -61,6 +61,8 @@
     EXPECT_STREQ(s2.c_str(), "s2");
     hidl_string s2a(nullptr); // copy constructor from null cstr
     EXPECT_STREQ("", s2a);
+    s2a = nullptr; // = from nullptr cstr
+    EXPECT_STREQ(s2a.c_str(), "");
     hidl_string s3 = hidl_string("s3"); // move =
     EXPECT_STREQ(s3.c_str(), "s3");
     hidl_string s4 = hidl_string("12345", 3); // copy constructor from cstr w/ length