Merge "midi: Use std::optional instead of std::unique_ptr" am: e20137fccd am: 85056fca49 am: 54ea8228cc

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

Change-Id: I295a0179516cfc8368cebcc9ed748846c82d04f5
This commit is contained in:
Jooyung Han
2020-11-25 06:33:37 +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());