Strip comments

Author: aurora0x27

src/Feature/Hover.cpp

@@ -168,7 +168,8 @@ static std::string get_document(CompilationUnit& unit,
     if(!comment) {
         return "";
     }
-    auto raw_string = comment->getRawText(Ctx.getSourceManager()).str();
+    auto raw_string = comment->getFormattedText(Ctx.getSourceManager(), Ctx.getDiagnostics());
+    LOG_WARN("Got comment:\n```\n{}\n```\n", raw_string);
     return "";
 }
 
@@ -426,7 +427,9 @@ static std::optional<Hover> hover(CompilationUnit& unit,
                                   const config::HoverOptions& opt) {
     // TODO: Hover for type
     // TODO: Add source code
-    return Hover{.kind = SymbolKind::Type, .name = ty.getAsString()};
+    auto& ctx = unit.context();
+    auto pp = ctx.getPrintingPolicy();
+    return Hover{.kind = SymbolKind::Type, .name = print_type(ctx, ty, pp, opt)};
 }
 
 static std::optional<Hover> hover(CompilationUnit& unit,

tests/unit/Feature/HoverTests.cpp

@@ -245,6 +245,72 @@ int main() {
     }
 };
 
+TEST_CASE(Comments) {
+    constexpr auto code = R"CODE(
+/// This is line comment
+/// This comment line 2
+///     This comment line 3 with indent
+/// This comment line 4
+static void fu$(pos_0)nc() {
+
+}
+
+// This is line comment
+// This comment line 2
+//     This comment line 3 with indent
+// This comment line 4
+static void fu$(pos_1)nc1() {
+
+}
+
+/*
+ * This is block comment
+ * This comment line 2
+ *     This comment line 3 with indent
+ * This comment line 4
+ */
+static void fu$(pos_2)nc2() {
+
+}
+
+/**
+ * This is block comment
+ * This comment line 2
+ *     This comment line 3 with indent
+ * This comment line 4
+ **/
+static void fu$(pos_3)nc3() {
+
+}
+
+/********************************************
+ * This is block comment
+ * This comment line 2
+ *     This comment line 3 with indent
+ * This comment line 4
+ *******************************************/
+static void fu$(pos_4)nc4() {
+
+}
+    )CODE";
+
+    auto annotation = AnnotatedSource::from(code);
+    tester.add_main("main.cpp", annotation.content);
+    tester.compile();
+    ASSERT_TRUE(tester.unit.has_value());
+    const unsigned count = annotation.offsets.size();
+    for(unsigned i = 0; i < count; ++i) {
+        unsigned offset = annotation.offsets[std::format("pos_{}", i)];
+        auto HI = clice::feature::hover(*tester.unit, offset, {});
+        // if(HI.has_value()) {
+        //     auto msg = HI->display({});
+        //     std::println("```\n{}```\n", *msg);
+        // } else {
+        //     std::println("No hover info");
+        // }
+    }
+}
+
 TEST_CASE(Namespace) {
     run(R"cpp(
 namespace A {