This probably isn't the *root* cause of why an executor is leaked,
but by cutting off this cyclic references, it may make it a bit easier
to track down leaks of an executor.
Max Brunsfeld
created
3788efe
Clean up guest connection states correctly when a collaborator disconnects
Click to expand commit body
This bug was caught by running the executor until parked after tests.
Co-Authored-By: Nathan Sobo <nathan@zed.dev>
Max Brunsfeld
and
Nathan Sobo
created
c09921e
Clear the executor at the end of a test by running until it is parked
Max Brunsfeld
created
426e0e3
Flush effects when dropping a TestAppContext
Click to expand commit body
This way, at the end of a test, dropped entities will be removed, and will drop
the handles that they hold.
Max Brunsfeld
created
7d53e37
Start work on detecting leaked handles in tests
Click to expand commit body
For now, just track models. Tests fail because we don't
yet clear the app contexts at the right time.
Co-Authored-By: Nathan Sobo <nathan@zed.dev>
2111ec0
Make `SearchProject` a `Foreground` message
Click to expand commit body
However, the randomized integration test is still failing:
```
ITERATIONS=100000 SEED=3027 OPERATIONS=200 cargo test --release test_random --package=zed-server -- --nocapture
```
Co-Authored-By: Nathan Sobo <nathan@zed.dev>
Antonio Scandurra
and
Nathan Sobo
created
400a2fc
Don't use a bounded channel for signaling that buffers have been opened
Click to expand commit body
Blocking the sender could halt deserialization for no reason if nobody
is consuming the notifications.
Antonio Scandurra
created
1313ca8
Don't delete buffer state when calling `get_open_buffer`
Click to expand commit body
...as we might be in the process of completing a request that could
open a buffer. This was causing a failure in the randomized integration
test.
Antonio Scandurra
created
5f7a759
Add project-wide search to randomized integration test
Antonio Scandurra
created
abdfdcd
Include buffer's deferred ops when computing `has_buffered_operations`
cb230ad
Re-activate the most recently-activated project search on cmd-shift-F
Click to expand commit body
This commits adds the beginnings of an application state facility as a non-static place to store the most recently-activated search for each project.
I also store workspace items by descending order of their entity id so that we always fetch the newest item of a given type when calling `Workspace::item_of_type`.
Nathan Sobo
created
1ddae2a
Focus the project find results editor on a tab in the query editor
Also: Clear the selection when we focus the results editor because we continue to render the selection even when the editor isn't focused and it looks awkward. Another approach we could take is to not render selections for non-focused editors, either always or with an option. But considering that we select all anyways next time we return focus to the query editor, I think this is ok for now.
Nathan Sobo
created
e96d0a9
Activate the *newest* existing project find view on cmd-shift-F
Nathan Sobo
created
2f42776
Allow a new search to be created with cmd-enter
Click to expand commit body
This replaces the `cmd-alt-shift-F` binding to open a new search. Instead, you can preserve the existing search results by entering a query and then hitting `cmd-enter` instead of `enter`. This opens a new project find view and restores the previous view to whatever query it was previously displaying. It's a bit strange, but I don't want to rely on splitting as the only way of creating multiple sets of search results.
Nathan Sobo
created
ae1a46a
Render a magnifier icon and the query in project search tab
Click to expand commit body
Also: Wire up events so the modified status updates correctly.
Nathan Sobo
created
a78fe4e
Don't focus results editor on `cmd-shift-f` when there are no results
f6b7cbd
Always open a new project find on `alt-cmd-shift-F`
Nathan Sobo
created
60710fa
Only store one nav history entry when opening excerpts
Click to expand commit body
Also: Introduce the ability to disable and enable the nav history directly. This allows us to explicitly push an entry when opening excerpts and then disable all pushes as we open individual buffers.
Also: Remove special handling for alt-shift-D binding in diagnostics view that opens excerpts. Rely on alt-enter in all multi-buffers instead.
Co-Authored-By: Max Brunsfeld <maxbrunsfeld@gmail.com>
Nathan Sobo
and
Max Brunsfeld
created
dea40c5
Don't show buffer search UI on ProjectSearchView
Click to expand commit body
Co-Authored-By: Max Brunsfeld <maxbrunsfeld@gmail.com>
Nathan Sobo
and
Max Brunsfeld
created
ed6c8b1
Allow actions to be propagated from nested ViewContexts
Click to expand commit body
Co-Authored-By: Max Brunsfeld <maxbrunsfeld@gmail.com>
8dce91b
Upgrade time crates to silence warning on Rust 1.59
Max Brunsfeld
created
9a97588
Eliminate RwLock around LanguageServer's outbound message channel
Click to expand commit body
We observed a deadlock when quitting zed. The main thread was attempting
to acquire a write lock to this outbound message sender. We weren't able
to understand exactly how this occurred, but we removed the use of a
lock there, so this shouldn't happen anymore.
Co-Authored-By: Antonio Scandurra <me@as-cii.com>
Max Brunsfeld
and
Antonio Scandurra
created
368301f
Reuse a previous project find whenever possible
Click to expand commit body
Co-Authored-By: Max Brunsfeld <max@zed.dev>
Antonio Scandurra
and
Max Brunsfeld
created
e278c42
Don't assume that cloning on split will reuse the same underlying model
Click to expand commit body
Co-Authored-By: Max Brunsfeld <max@zed.dev>
Antonio Scandurra
and
Max Brunsfeld
created
7123407
Don't share query editor state after project find has been split
Click to expand commit body
Co-Authored-By: Nathan Sobo <nathan@zed.dev>
Antonio Scandurra
and
Nathan Sobo
created
1e04411
Don't focus query editor if there are matches on tab switch
Antonio Scandurra
created
b506db7
Use the new split pane's navigation history when cloning an item