From abe09bfd763b4e37aa8d76884412a755ef7be5dd Mon Sep 17 00:00:00 2001 From: Christian Kandeler Date: Wed, 26 Jan 2022 12:40:12 +0100 Subject: ClangCodeModel: Adjust cursor for following destructor symbols ... with clangd. More specifically, for the case where we switch between declaration and definition of a destructor. Since clangd expects the cursor before the "~" character, we move it there. Fixes: QTCREATORBUG-26809 Change-Id: I8ea6a060bea3545c963cb8ebbe2d3ae3a18da2d5 Reviewed-by: Reviewed-by: Christian Stenger --- src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp') diff --git a/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp b/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp index 8cfff10c658..81b410a3cda 100644 --- a/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp +++ b/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp @@ -263,10 +263,6 @@ F2TestCase::F2TestCase(CppEditorAction action, QSKIP("fuzzy matching is not supposed to work with clangd"); // TODO: Implement fallback as we do with libclang if (tag == "baseClassFunctionIntroducedByUsingDeclaration") QSKIP("clangd points to the using declaration"); - if (tag == "classDestructor" || tag == "fromDestructorDefinitionSymbol" - || tag == "fromDestructorBody") { - QSKIP("clangd wants the cursor before the ~ character"); - } if (curTestName == "testFollowClassOperatorInOp") QSKIP("clangd goes to operator name first"); } @@ -438,6 +434,10 @@ F2TestCase::F2TestCase(CppEditorAction action, } else { currentTextEditor->convertPosition(targetTestFile->m_targetCursorPosition, &expectedLine, &expectedColumn); + if (useClangd && (tag == "classDestructor" || tag == "fromDestructorDefinitionSymbol" + || tag == "fromDestructorBody")) { + --expectedColumn; // clangd goes before the ~, built-in code model after + } } // qDebug() << "Expected line:" << expectedLine; // qDebug() << "Expected column:" << expectedColumn; -- cgit v1.2.3