Merge pull request #321 from charmbracelet/inefassign

Kujtim Hoxha created 3 months ago

fix: remove unused constants, fix inefficient assign

Change summary

internal/lsp/watcher/ulimit_darwin.go              | 3 +--
internal/tui/components/completions/completions.go | 5 -----
2 files changed, 1 insertion(+), 7 deletions(-)

Detailed changes

internal/lsp/watcher/ulimit_darwin.go

@@ -5,13 +5,12 @@ package watcher
 import "syscall"
 
 func Ulimit() (uint64, error) {
-	var currentLimit uint64 = 0
 	var rLimit syscall.Rlimit
 	err := syscall.Getrlimit(syscall.RLIMIT_NOFILE, &rLimit)
 	if err != nil {
 		return 0, err
 	}
-	currentLimit = rLimit.Cur
+	currentLimit := rLimit.Cur
 	rLimit.Cur = rLimit.Max / 10 * 8
 	err = syscall.Setrlimit(syscall.RLIMIT_NOFILE, &rLimit)
 	if err != nil {

internal/tui/components/completions/completions.go

@@ -71,11 +71,6 @@ type completionsCmp struct {
 	query string // The current filter query
 }
 
-const (
-	maxCompletionsWidth = 80 // Maximum width for the completions popup
-	minCompletionsWidth = 20 // Minimum width for the completions popup
-)
-
 func New() Completions {
 	completionsKeyMap := DefaultKeyMap()
 	keyMap := list.DefaultKeyMap()