Merge "midi: Use std::optional instead of std::unique_ptr" am: e20137fccd

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1502510

Change-Id: I11bdd3b08fa2d83752010a11461d5051678ccbfd
This commit is contained in:
Jooyung Han
2020-11-25 04:22:35 +00:00
committed by Automerger Merge Worker

View File

@@ -90,13 +90,13 @@ status_t MidiDeviceInfo::readFromParcel(const Parcel* parcel) {
status_t MidiDeviceInfo::readStringVector(
const Parcel* parcel, Vector<String16> *vectorPtr, size_t defaultLength) {
std::unique_ptr<std::vector<std::unique_ptr<String16>>> v;
std::optional<std::vector<std::optional<String16>>> v;
status_t result = parcel->readString16Vector(&v);
if (result != OK) return result;
vectorPtr->clear();
if (v.get() != nullptr) {
if (v) {
for (const auto& iter : *v) {
if (iter.get() != nullptr) {
if (iter) {
vectorPtr->push_back(*iter);
} else {
vectorPtr->push_back(String16());