Utils: remove OptionalLineColumn

LineColumn already has a isValid() that is sufficient for the use case.

Change-Id: I7f6e1d64b66a9af05d74ce0ef45717265dc28ed3
Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>

3 files changed, 5 insertions, 10 deletions

diff --git a/src/libs/utils/linecolumn.h b/src/libs/utils/linecolumn.h
index 457ebd6a51..922c981d79 100644
--- a/src/libs/utils/linecolumn.h
+++ b/src/libs/utils/linecolumn.h
@@ -39,8 +39,6 @@ public:
     int column = -1;
 };
 
-using OptionalLineColumn = std::optional<LineColumn>;
-
 } // namespace Utils
 
 Q_DECLARE_METATYPE(Utils::LineColumn)
diff --git a/src/libs/utils/textutils.cpp b/src/libs/utils/textutils.cpp
index 8b52d51d2c..55e8a736d0 100644
--- a/src/libs/utils/textutils.cpp
+++ b/src/libs/utils/textutils.cpp
@@ -53,16 +53,13 @@ bool convertPosition(const QTextDocument *document, int pos, int *line, int *col
     }
 }
 
-OptionalLineColumn convertPosition(const QTextDocument *document, int pos)
+LineColumn convertPosition(const QTextDocument *document, int pos)
 {
-    OptionalLineColumn optional;
-
-    QTextBlock block = document->findBlock(pos);
-
+    const QTextBlock block = document->findBlock(pos);
     if (block.isValid())
-        optional.emplace(block.blockNumber() + 1, pos - block.position());
+        return {block.blockNumber() + 1, pos - block.position()};
 
-    return optional;
+    return {};
 }
 
 int positionInText(const QTextDocument *textDocument, int line, int column)
diff --git a/src/libs/utils/textutils.h b/src/libs/utils/textutils.h
index ce8a450302..573dc7aa1d 100644
--- a/src/libs/utils/textutils.h
+++ b/src/libs/utils/textutils.h
@@ -69,7 +69,7 @@ QTCREATOR_UTILS_EXPORT bool convertPosition(const QTextDocument *document,
                                             int pos,
                                             int *line, int *column);
 QTCREATOR_UTILS_EXPORT
-OptionalLineColumn convertPosition(const QTextDocument *document, int pos);
+LineColumn convertPosition(const QTextDocument *document, int pos);
 
 // line and column are 1-based
 QTCREATOR_UTILS_EXPORT int positionInText(const QTextDocument *textDocument, int line, int column);