try to improve the signature of a constructor call if the symbol only…

… gives us "unit" as a return type (#1312)
ionide · Aug 18, 2024 · 2e2fd24 · 2e2fd24
1 parent fc79e99
commit 2e2fd24
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/FsAutoComplete.Core/SignatureFormatter.fs b/src/FsAutoComplete.Core/SignatureFormatter.fs
@@ -314,8 +314,11 @@ module SignatureFormatter =
         retType
       //A ctor with () parameters seems to be a list with an empty list.
       // Also abstract members and abstract member overrides with one () parameter seem to be a list with an empty list.
-      elif func.IsConstructor || (func.IsMember && (not func.IsPropertyGetterMethod)) then
-        modifiers + ": unit -> " ++ retType
+      elif func.IsConstructor then
+        let retType = if retType = "unit" then func.DisplayNameCore else retType
+        modifiers + ": unit ->" ++ retType
+      elif func.IsMember && (not func.IsPropertyGetterMethod) then
+        modifiers + ": unit ->" ++ retType
       else
         modifiers ++ functionName + ":" ++ retType //Value members seems to be a list with an empty list
     | [ [ p ] ] when maybeGetter && formatParameter p = "unit" -> //Member or property with only getter