Merge pull request #434 from thetarnav/inlay-hints-comma

Remove comma from first param default value inlay hint

1 files changed, 14 insertions, 15 deletions

diff --git a/src/server/inlay_hints.odin b/src/server/inlay_hints.odin
index 55cd486..3f8d181 100644
--- a/src/server/inlay_hints.odin
+++ b/src/server/inlay_hints.odin
@@ -128,21 +128,20 @@ get_inlay_hints :: proc(
 							position = call_range.end
 							position.character -= 1
 
-							has_trailing_comma: bool;{
-								if !has_added_default {
-									till_end := string(
-										document.text[:call.close.offset],
-									)
-									trailing_loop: #reverse for ch in till_end {
-										switch ch {
-										case ' ', '\t', '\n':
-										case ',':
-											has_trailing_comma = true
-											break trailing_loop
-										case:
-											break trailing_loop
-										}
+							needs_leading_comma := i > 0
+
+							if !has_added_default && needs_leading_comma {
+								till_end := string(
+									document.text[:call.close.offset],
+								)
+								#reverse for ch in till_end {
+									switch ch {
+									case ' ', '\t', '\n':
+										continue
+									case ',':
+										needs_leading_comma = false
 									}
+									break
 								}
 							}
 
@@ -150,7 +149,7 @@ get_inlay_hints :: proc(
 								kind     = .Parameter,
 								label    = fmt.tprintf(
 									"%s %v := %v",
-									has_trailing_comma ? "" : ",",
+									needs_leading_comma ? "," : "",
 									label,
 									value,
 								),