diff options
author | Jooyung Han <jooyung@google.com> | 2020-11-18 12:17:50 +0900 |
---|---|---|
committer | Jooyung Han <jooyung@google.com> | 2020-11-18 13:12:31 +0900 |
commit | aac823d704ad0679a2a7c62285e9475ed782f819 (patch) | |
tree | 30170b5a9bcdb32d8391c807acab1f55ba23c442 /media/native/midi/MidiDeviceInfo.cpp | |
parent | 5bc68e02a01c2621aa64e2d410a2caa6c97bc190 (diff) |
midi: Use std::optional instead of std::unique_ptr
Since Android R, Parcel provides read/write methods for "@nullable"
values using std::optional.
Bug: 149784838
Test: m
Change-Id: I5c6a87072ced2f2a3a16df374244bb0aff5f9cb3
Diffstat (limited to 'media/native/midi/MidiDeviceInfo.cpp')
-rw-r--r-- | media/native/midi/MidiDeviceInfo.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/media/native/midi/MidiDeviceInfo.cpp b/media/native/midi/MidiDeviceInfo.cpp index ac68d26c935b..8a573fba322b 100644 --- a/media/native/midi/MidiDeviceInfo.cpp +++ b/media/native/midi/MidiDeviceInfo.cpp @@ -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()); |