From 5891420e6848e86a069dc6b7b73f175e22388f4a Mon Sep 17 00:00:00 2001 From: Kazu Hirata Date: Sat, 3 Dec 2022 11:54:46 -0800 Subject: [clang] Use std::nullopt instead of None (NFC) This patch mechanically replaces None with std::nullopt where the compiler would warn if None were deprecated. The intent is to reduce the amount of manual work required in migrating from Optional to std::optional. This is part of an effort to migrate from llvm::Optional to std::optional: https://discourse.llvm.org/t/deprecating-llvm-optional-x-hasvalue-getvalue-getvalueor/63716 --- clang/lib/ExtractAPI/ExtractAPIConsumer.cpp | 4 ++-- .../lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp | 14 +++++++------- 2 files changed, 9 insertions(+), 9 deletions(-) (limited to 'clang/lib/ExtractAPI') diff --git a/clang/lib/ExtractAPI/ExtractAPIConsumer.cpp b/clang/lib/ExtractAPI/ExtractAPIConsumer.cpp index 2ab1828..7c56818 100644 --- a/clang/lib/ExtractAPI/ExtractAPIConsumer.cpp +++ b/clang/lib/ExtractAPI/ExtractAPIConsumer.cpp @@ -157,7 +157,7 @@ Optional getRelativeIncludeName(const CompilerInstance &CI, Rule.match(File, &Matches); // Returned matches are always in stable order. if (Matches.size() != 4) - return None; + return std::nullopt; return path::convert_to_slash( (Matches[1].drop_front(Matches[1].rfind('/') + 1) + "/" + @@ -172,7 +172,7 @@ Optional getRelativeIncludeName(const CompilerInstance &CI, } // Couldn't determine a include name, use full path instead. - return None; + return std::nullopt; } struct LocationFileChecker { diff --git a/clang/lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp b/clang/lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp index b74cf5b4..91dc252 100644 --- a/clang/lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp +++ b/clang/lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp @@ -60,7 +60,7 @@ void serializeArray(Object &Paren, StringRef Key, Optional Array) { /// the semantic version representation of \p V. Optional serializeSemanticVersion(const VersionTuple &V) { if (V.empty()) - return None; + return std::nullopt; Object Version; Version["major"] = V.getMajor(); @@ -146,7 +146,7 @@ Object serializeSourceRange(const PresumedLoc &BeginLoc, /// an \c Array containing the formatted availability information. Optional serializeAvailability(const AvailabilitySet &Availabilities) { if (Availabilities.isDefault()) - return None; + return std::nullopt; Array AvailabilityArray; @@ -232,7 +232,7 @@ Object serializeIdentifier(const APIRecord &Record, Language Lang) { /// formatted lines. Optional serializeDocComment(const DocComment &Comment) { if (Comment.empty()) - return None; + return std::nullopt; Object DocComment; Array LinesArray; @@ -284,7 +284,7 @@ Optional serializeDocComment(const DocComment &Comment) { /// declaration fragments array. Optional serializeDeclarationFragments(const DeclarationFragments &DF) { if (DF.getFragments().empty()) - return None; + return std::nullopt; Array Fragments; for (const auto &F : DF.getFragments()) { @@ -412,7 +412,7 @@ Optional serializeFunctionSignatureMixinImpl(const RecordTy &Record, std::true_type) { const auto &FS = Record.Signature; if (FS.empty()) - return None; + return std::nullopt; Object Signature; serializeArray(Signature, "returns", @@ -436,7 +436,7 @@ Optional serializeFunctionSignatureMixinImpl(const RecordTy &Record, template Optional serializeFunctionSignatureMixinImpl(const RecordTy &Record, std::false_type) { - return None; + return std::nullopt; } /// Serialize the function signature field, as specified by the @@ -501,7 +501,7 @@ template Optional SymbolGraphSerializer::serializeAPIRecord(const RecordTy &Record) const { if (shouldSkip(Record)) - return None; + return std::nullopt; Object Obj; serializeObject(Obj, "identifier", -- cgit v1.1