Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | allow rustfmt to reorder imports | Aleksey Kladov | 2019-07-04 | 13 | -58/+52 |
| | | | | | | This wasn't a right decision in the first place, the feature flag was broken in the last rustfmt release, and syntax highlighting of imports is more important anyway | ||||
* | Move memory usage statistics to ra_prof | Aleksey Kladov | 2019-06-30 | 1 | -1/+1 |
| | |||||
* | Swallow expected `rustfmt` errors | Ryan Cumming | 2019-06-26 | 1 | -10/+25 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | My workflow in Visual Studio Code + Rust Analyzer has become: 1. Make a change to Rust source code using all the analysis magic 2. Save the file to trigger `cargo watch`. I have format on save enabled for all file types so this also runs `rustfmt` 3. Fix any diagnostics that `cargo watch` finds Unfortunately if the Rust source has any syntax errors the act of saving will pop up a scary "command has failed" message and will switch to the "Output" tab to show the `rustfmt` error and exit code. I did a quick survey of what other Language Servers do in this case. Both the JSON and TypeScript servers will swallow the error and return success. This is consistent with how I remember my workflow in those languages. The syntax error will show up as a diagnostic so it should be clear why the file isn't formatting. I checked the `rustfmt` source code and while it does distinguish "parse errors" from "operational errors" internally they both result in exit status of 1. However, more catastrophic errors (missing `rustfmt`, SIGSEGV, etc) will return 127+ error codes which we can distinguish from a normal failure. This changes our handler to log an info message and feign success if `rustfmt` exits with status 1. Another option I considered was only swallowing the error if the formatting request came from format-on-save. However, the Language Server Protocol doesn't seem to distinguish those cases. | ||||
* | Bump cargo_metadata, ena, flexi_logger | kjeremy | 2019-06-20 | 1 | -1/+1 |
| | |||||
* | reuse AnalysisHost in batch analysis | Aleksey Kladov | 2019-06-15 | 1 | -1/+1 |
| | |||||
* | re-enable backtraces on panic | Aleksey Kladov | 2019-06-15 | 1 | -2/+1 |
| | |||||
* | cargo format | Muhammad Mominul Huque | 2019-06-15 | 1 | -7/+2 |
| | |||||
* | Get rid of failure: ra_lsp_server & ra_project_model | Muhammad Mominul Huque | 2019-06-14 | 5 | -23/+25 |
| | |||||
* | Temp fix for slow onEnter issue | Aleksey Kladov | 2019-06-13 | 1 | -1/+2 |
| | | | | | | | | | | The issue was windows specific -- cancellation caused collection of bracktraces at some point, and that was slow on windows. The proper fix here is to make sure that we don't collect bracktraces unnecessary (which we currently do due to failure), but, as a temporary fix, let's just not force their collection in the first place! | ||||
* | make LRU cache configurable | Aleksey Kladov | 2019-06-12 | 3 | -6/+18 |
| | |||||
* | make Docs handing more ideomatic | Aleksey Kladov | 2019-06-08 | 2 | -17/+8 |
| | |||||
* | Fix clippy::or_fun_call | Alan Du | 2019-06-04 | 1 | -1/+1 |
| | |||||
* | Fix clippy::identity_conversion | Alan Du | 2019-06-04 | 3 | -20/+15 |
| | |||||
* | Fix clippy::unused_mut | Alan Du | 2019-06-04 | 1 | -1/+1 |
| | |||||
* | Fix clippy::unnecessary_mut_passed | Alan Du | 2019-06-04 | 1 | -7/+2 |
| | |||||
* | Fix clippy::single_match | Alan Du | 2019-06-04 | 1 | -4/+3 |
| | |||||
* | rename | Aleksey Kladov | 2019-06-01 | 6 | -80/+86 |
| | |||||
* | move subs inside | Aleksey Kladov | 2019-06-01 | 1 | -4/+2 |
| | |||||
* | use sync queries for join lines and friends | Aleksey Kladov | 2019-05-31 | 1 | -5/+11 |
| | |||||
* | add sync requests | Aleksey Kladov | 2019-05-31 | 2 | -43/+56 |
| | |||||
* | cleanup | Aleksey Kladov | 2019-05-31 | 1 | -39/+42 |
| | |||||
* | cleanup | Aleksey Kladov | 2019-05-31 | 1 | -35/+48 |
| | |||||
* | simplify | Aleksey Kladov | 2019-05-31 | 1 | -51/+52 |
| | |||||
* | move completed requests to a separate file | Aleksey Kladov | 2019-05-31 | 5 | -80/+114 |
| | |||||
* | simplify | Aleksey Kladov | 2019-05-31 | 1 | -3/+3 |
| | |||||
* | introduce constant | Aleksey Kladov | 2019-05-31 | 1 | -7/+13 |
| | |||||
* | minor | Aleksey Kladov | 2019-05-31 | 1 | -1/+1 |
| | |||||
* | update ra_ide_api to use builtins | Aleksey Kladov | 2019-05-30 | 1 | -0/+1 |
| | |||||
* | :arrow_up: parking_lot | Aleksey Kladov | 2019-05-30 | 1 | -1/+1 |
| | |||||
* | bump timeout for CI | Aleksey Kladov | 2019-05-29 | 1 | -1/+1 |
| | |||||
* | less noisy status | Aleksey Kladov | 2019-05-29 | 1 | -1/+1 |
| | |||||
* | optimization: cancel backlog in onEnter | Aleksey Kladov | 2019-05-29 | 2 | -3/+16 |
| | |||||
* | add latest requests to status page | Aleksey Kladov | 2019-05-29 | 3 | -12/+67 |
| | |||||
* | log the actual time of requests | Aleksey Kladov | 2019-05-29 | 1 | -16/+31 |
| | |||||
* | trigger garbage collection *after* requests, not before | Aleksey Kladov | 2019-05-29 | 1 | -2/+5 |
| | |||||
* | more perf logging | Aleksey Kladov | 2019-05-29 | 1 | -3/+8 |
| | |||||
* | silnce profiling in tests | Aleksey Kladov | 2019-05-29 | 1 | -1/+2 |
| | |||||
* | Merge #1334 | bors[bot] | 2019-05-27 | 2 | -1/+82 |
|\ | | | | | | | | | | | | | | | 1334: check for cancellation during macro expansion r=matklad a=matklad closes #1331 Co-authored-by: Aleksey Kladov <[email protected]> | ||||
| * | check cancellation when expanding macros | Aleksey Kladov | 2019-05-27 | 1 | -3/+2 |
| | | |||||
| * | enable profiling in tests | Aleksey Kladov | 2019-05-27 | 2 | -1/+83 |
| | | |||||
* | | rename stray id field | Pascal Hertleif | 2019-05-27 | 2 | -2/+2 |
| | | |||||
* | | make it build again | Pascal Hertleif | 2019-05-27 | 1 | -1/+1 |
| | | |||||
* | | Semantic highlighting spike | Pascal Hertleif | 2019-05-27 | 2 | -1/+6 |
|/ | | | | | | | | | | Very simple approach: For each identifier, set the hash of the range where it's defined as its 'id' and use it in the VSCode extension to generate unique colors. Thus, the generated colors are per-file. They are also quite fragile, and I'm not entirely sure why. Looks like we need to make sure the same ranges aren't overwritten by a later request? | ||||
* | add profile calls to real-time requests | Aleksey Kladov | 2019-05-27 | 1 | -0/+5 |
| | |||||
* | Added local macro goto | Lenard Pratt | 2019-05-04 | 1 | -0/+1 |
| | |||||
* | Basic resolution for ADT | kjeremy | 2019-04-23 | 3 | -2/+23 |
| | |||||
* | :arrow_up: lsp | Aleksey Kladov | 2019-04-21 | 1 | -1/+1 |
| | |||||
* | switch to official extend selection API | Aleksey Kladov | 2019-04-21 | 5 | -3/+72 |
| | |||||
* | cleanup cancellation | Aleksey Kladov | 2019-04-17 | 1 | -10/+5 |
| | | | | | Now that we explicitelly exit the reading loop on exit notification, we can assume that the sender is always alive | ||||
* | add a couple of profiling points | Aleksey Kladov | 2019-04-14 | 1 | -0/+3 |
| |