Don't hang local x = function assignments

[JohnnyMorganz]

Fixes #508

[codecov]

Codecov Report

Merging #512 (2704951) into master (51fdff4) will increase coverage by 97.64%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master     #512       +/-   ##
===========================================
+ Coverage        0   97.64%   +97.64%     
===========================================
  Files           0       14       +14     
  Lines           0     5354     +5354     
===========================================
+ Hits            0     5228     +5228     
- Misses          0      126      +126     

Impacted Files	Coverage Δ
src/formatters/assignment.rs	97.88% <100.00%> (ø)
src/formatters/block.rs	97.69% <0.00%> (ø)
src/formatters/stmt.rs	98.77% <0.00%> (ø)
src/lib.rs	96.29% <0.00%> (ø)
src/formatters/table.rs	98.69% <0.00%> (ø)
src/formatters/trivia.rs	100.00% <0.00%> (ø)
src/formatters/general.rs	96.66% <0.00%> (ø)
src/formatters/mod.rs	100.00% <0.00%> (ø)
src/formatters/functions.rs	98.61% <0.00%> (ø)
... and 5 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 51fdff4...2704951. Read the comment docs.

[github-actions]

Repo Comparison Test

diff --git ORI/neovim/runtime/lua/vim/lsp.lua ALT/neovim/runtime/lua/vim/lsp.lua
index 61586ca..5ffb1ab 100644
--- ORI/neovim/runtime/lua/vim/lsp.lua
+++ ALT/neovim/runtime/lua/vim/lsp.lua
@@ -1385,17 +1385,16 @@ end
 --- Notify all attached clients that a buffer has changed.
 local text_document_did_change_handler
 do
-  text_document_did_change_handler =
-    function(_, bufnr, changedtick, firstline, lastline, new_lastline)
-      -- Detach (nvim_buf_attach) via returning True to on_lines if no clients are attached
-      if tbl_isempty(all_buffer_active_clients[bufnr] or {}) then
-        return true
-      end
-      util.buf_versions[bufnr] = changedtick
-      local compute_change_and_notify =
-        changetracking.prepare(bufnr, firstline, lastline, new_lastline)
-      for_each_buffer_client(bufnr, compute_change_and_notify)
+  text_document_did_change_handler = function(_, bufnr, changedtick, firstline, lastline, new_lastline)
+    -- Detach (nvim_buf_attach) via returning True to on_lines if no clients are attached
+    if tbl_isempty(all_buffer_active_clients[bufnr] or {}) then
+      return true
     end
+    util.buf_versions[bufnr] = changedtick
+    local compute_change_and_notify =
+      changetracking.prepare(bufnr, firstline, lastline, new_lastline)
+    for_each_buffer_client(bufnr, compute_change_and_notify)
+  end
 end
 
 ---@private

[JohnnyMorganz]

Not the right fix - luau regression test shows we will make something worse, and the neovim test now goes over neovim's 100 char budget

[JohnnyMorganz]

Will revisit later. This isn't correct fix and the problem is looking a lot more involved.