From ca61b6c1ab7966cb7f3509c81439e5c673f5ae88 Mon Sep 17 00:00:00 2001 From: Tillmann Karras Date: Sat, 1 Aug 2020 01:28:21 +0100 Subject: [PATCH] DiscIO: make use of std::nullopt This also fixes a bogus GCC warning. --- Source/Core/DiscIO/Volume.h | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/Source/Core/DiscIO/Volume.h b/Source/Core/DiscIO/Volume.h index e176c5d72a..19bd968e53 100644 --- a/Source/Core/DiscIO/Volume.h +++ b/Source/Core/DiscIO/Volume.h @@ -58,15 +58,20 @@ public: std::optional ReadSwappedAndShifted(u64 offset, const Partition& partition) const { const std::optional temp = ReadSwapped(offset, partition); - return temp ? static_cast(*temp) << GetOffsetShift() : std::optional(); + if (!temp) + return std::nullopt; + return static_cast(*temp) << GetOffsetShift(); } virtual bool IsEncryptedAndHashed() const { return false; } virtual std::vector GetPartitions() const { return {}; } virtual Partition GetGamePartition() const { return PARTITION_NONE; } - virtual std::optional GetPartitionType(const Partition& partition) const { return {}; } + virtual std::optional GetPartitionType(const Partition& partition) const + { + return std::nullopt; + } std::optional GetTitleID() const { return GetTitleID(GetGamePartition()); } - virtual std::optional GetTitleID(const Partition& partition) const { return {}; } + virtual std::optional GetTitleID(const Partition& partition) const { return std::nullopt; } virtual const IOS::ES::TicketReader& GetTicket(const Partition& partition) const { return INVALID_TICKET;