diff options
| author | Christian Stenger <christian.stenger@qt.io> | 2021-05-28 09:56:24 +0200 |
|---|---|---|
| committer | Christian Stenger <christian.stenger@qt.io> | 2021-05-28 08:03:58 +0000 |
| commit | 95ff7a6c80c6f9e7edef1710771d40ba399842bf (patch) | |
| tree | 9dd14347af7127137a36b89d9ef2ac3c4633d9a4 /src | |
| parent | 9237f66bc9a0f4eeebe509748351235d132d27f4 (diff) | |
LSP: Fix logic to avoid nullptr access
Change-Id: I5511d266b52b1bc1763bdf3cd430f49f30ae3f37
Reviewed-by: David Schulz <david.schulz@qt.io>
Diffstat (limited to 'src')
| -rw-r--r-- | src/libs/languageserverprotocol/completion.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/libs/languageserverprotocol/completion.cpp b/src/libs/languageserverprotocol/completion.cpp index ca34629805..faeb237718 100644 --- a/src/libs/languageserverprotocol/completion.cpp +++ b/src/libs/languageserverprotocol/completion.cpp @@ -45,8 +45,9 @@ Utils::optional<MarkupOrString> CompletionItem::documentation() const Utils::optional<CompletionItem::InsertTextFormat> CompletionItem::insertTextFormat() const { Utils::optional<int> value = optionalValue<int>(insertTextFormatKey); - return value.has_value() ? Utils::nullopt - : Utils::make_optional(CompletionItem::InsertTextFormat(value.value())); + return value.has_value() + ? Utils::make_optional(CompletionItem::InsertTextFormat(value.value())) + : Utils::nullopt; } bool CompletionItem::isValid(ErrorHierarchy *error) const |