aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| | * Fix extra parentheses warningsLaurențiu Nicola2020-01-311-2/+2
| |/
| * Merge #2960bors[bot]2020-01-311-4/+6
| |\ | | | | | | | | | | | | | | | | | | | | | 2960: Small cleanup r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| | * Small cleanupAleksey Kladov2020-01-311-4/+6
| |/
| * Merge #2946bors[bot]2020-01-301-2/+1
| |\ | | | | | | | | | | | | | | | | | | | | | 2946: updating nvim_lsp usage for rust_analyzer r=matklad a=matu3ba The other option is invalid and a working version depends on [PR](https://github.com/neovim/nvim-lsp/pull/107) Co-authored-by: Jan <[email protected]>
| | * updating nvim_lsp usage for rust_analyzerJan2020-01-291-2/+1
| | |
| * | Merge #2955bors[bot]2020-01-302-697/+689
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2955: :arrow_up: rust r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| | * | Switch Cargo.lock to the new formatAleksey Kladov2020-01-301-696/+688
| | | |
| | * | :arrow_up: rustAleksey Kladov2020-01-301-1/+1
| | | |
| * | | Merge #2954bors[bot]2020-01-301-0/+3
| |\ \ \ | | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | 2954: Add a FIXME note r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| | * | Add a FIXME noteAleksey Kladov2020-01-301-0/+3
| |/ /
| * | Merge #2920bors[bot]2020-01-301-2/+18
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2920: Better handle illformed node id from metadata r=matklad a=edwin0cheng In some rare cases, deps node-id from cargo-metadata do not match its version-id, which cause a panic in `cargo-workspace.rs`. This PR try to ignore these ill-formed node id from `cargo-metadata`. An alternative is return `Err` in these cases but I think make it resilience is a better choice here. Related #2767 Co-authored-by: Edwin Cheng <[email protected]>
| | * | Ignore illform node id from metadataEdwin Cheng2020-01-281-2/+18
| | | |
| * | | Merge #2953bors[bot]2020-01-301-20/+20
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2953: Update crates r=kjeremy a=kjeremy Co-authored-by: Jeremy Kolb <[email protected]>
| | * | | Update cratesJeremy Kolb2020-01-301-20/+20
| |/ / /
| * | | Merge #2952bors[bot]2020-01-301-21/+8
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2952: Simplify fixture parsing r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| | * | | Simplify fixture parsingAleksey Kladov2020-01-301-21/+8
| |/ / /
| * | | Merge #2895bors[bot]2020-01-291-46/+71
| |\ \ \ | | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2895: Rewrite ra_prof's profile printing r=michalt a=michalt This changes the way we print things to first construct a mapping from events to the children and uses that mapping to actually print things. It should not change the actual output that we produce. The new approach two benefits: * It avoids a potential quadratic behavior of the previous approach. For instance, for a vector of N elements: ``` [Message{level: (N - 1)}, ..., Message{level: 1}, Message{level: 0}] ``` we would first do a linear scan to find entry with level 0, then another scan to find one with level 1, etc. * It makes it much easier to improve the output in the future, because we now pre-compute the children for each entry and can easily take that into account when printing. Signed-off-by: Michal Terepeta <[email protected]> Co-authored-by: Michal Terepeta <[email protected]>
| | * | A couple of small improvements to ra_prof printingMichal Terepeta2020-01-291-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | Based on suggestions from @matklad. Signed-off-by: Michal Terepeta <[email protected]>
| | * | Rewrite ra_prof's profile printingMichal Terepeta2020-01-221-46/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changes the way we print things to first construct a mapping from events to the children and uses that mapping to actually print things. It should not change the actual output that we produce. The new approach two benefits: * It avoids a potential quadratic behavior of the previous approach. For instance, for a vector of N elements: ``` [Message{level: (N - 1)}, ..., Message{level: 1}, Message{level: 0}] ``` we would first do a linear scan to find entry with level 0, then another scan to find one with level 1, etc. * It makes it much easier to improve the output in the future, because we now pre-compute the children for each entry and can easily take that into account when printing. Signed-off-by: Michal Terepeta <[email protected]>
| * | | Merge #2943bors[bot]2020-01-292-20/+53
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2943: Re-sync queries for memory usage measurnment r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| | * | | Re-sync queries for memory usage measurnmentAleksey Kladov2020-01-292-20/+53
| | | | |
| * | | | Merge #2942bors[bot]2020-01-291-29/+17
| |\ \ \ \ | | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2942: Properly select a target for auto importing r=matklad a=SomeoneToIgnore Fixes https://github.com/rust-analyzer/rust-analyzer/issues/2932 The corresponding test has the caret placed in the beggining of the document despite the import inserted, but I don't thing I should fix it here: * in real life, there's some text written before the import and for those cases the caret behaves normally * it's a separate functionality that needs to be refactored anyway later (the `auto_import_text_edit`) Co-authored-by: Kirill Bulatov <[email protected]>
| | * | | Properly select a target for auto importingKirill Bulatov2020-01-291-29/+17
| |/ / /
| * | | Merge #2941bors[bot]2020-01-292-38/+45
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2941: Freshen Architecture.md document r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| | * | | Freshen Architecture.md documentAleksey Kladov2020-01-292-38/+45
| | | | |
| * | | | Merge #2940bors[bot]2020-01-292-48/+38
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2940: Freshen dev docs a tiny bits r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| | * | | Freshen dev docs a tiny bitsAleksey Kladov2020-01-292-48/+38
| | | | |
| * | | | Merge #2939bors[bot]2020-01-291-20/+19
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2939: Simplify fixture parsing r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| | * | | Simplify fixture parsingAleksey Kladov2020-01-291-20/+19
| | | | |
| * | | | Merge #2938bors[bot]2020-01-291-1/+1
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2938: Fix long loop timeout r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| | * | | Fix long loop timeoutAleksey Kladov2020-01-291-1/+1
| |/ / /
| * | | Merge #2937bors[bot]2020-01-292-5/+26
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2937: Parse cargo output a line at a time. r=kiljacken a=kiljacken We previously used serde's stream deserializer to read json blobs from the cargo output. It has an issue though: If the deserializer encounters invalid input, it gets stuck reporting the same error again and again because it is unable to foward over the input until it reaches a new valid object. Reading a line at a time and manually deserializing fixes this issue, because cargo makes sure to only outpu one json blob per line, so should we encounter invalid input, we can just skip a line and continue. The main reason this would happen is stray printf-debugging in procedural macros, so we still report that an error occured, but we handle it gracefully now. Fixes #2935 Co-authored-by: Emil Lauridsen <[email protected]>
| | * | | Change error output to make a bit more senseEmil Lauridsen2020-01-291-2/+6
| | | | |
| | * | | Parse cargo output a line at a time.Emil Lauridsen2020-01-292-5/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We previously used serde's stream deserializer to read json blobs from the cargo output. It has an issue though: If the deserializer encounters invalid input, it gets stuck reporting the same error again and again because it is unable to foward over the input until it reaches a new valid object. Reading a line at a time and manually deserializing fixes this issue, because cargo makes sure to only outpu one json blob per line, so should we encounter invalid input, we can just skip a line and continue. The main reason this would happen is stray printf-debugging in procedural macros, so we still report that an error occured, but we handle it gracefully now. Fixes #2935
| * | | | Tweak readmeAleksey Kladov2020-01-291-3/+2
| | | | |
| * | | | Merge pull request #2936 from matklad/better-releaseAleksey Kladov2020-01-291-2/+4
| |\ \ \ \ | | |/ / / | |/| | | Make sure release uses the release branch, and not master
| | * | | Make sure release uses the release branch, and not masterAleksey Kladov2020-01-291-2/+4
| | | | |
| * | | | Merge #2931bors[bot]2020-01-292-24/+56
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2931: Added documentation to test_utils r=matklad a=Veetaha Added some doc comments to test_utils functions while studying this crate. They should be all stable enough to document them. Also some minor code relocation in `parse_fixture()` closer to its usage according to the advice of @matklad. Co-authored-by: Veetaha <[email protected]>
| | * | | | test_utils: move flush!() to its usage as per conversation with @matkladVeetaha2020-01-291-13/+13
| | | | | |
| | * | | | test_utils: updated documentation and some typosVeetaha2020-01-292-11/+43
| | | | | |
| * | | | | Merge #2934bors[bot]2020-01-291-8/+28
| |\ \ \ \ \ | | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2934: Complain loudly if the main loop is blocked r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| | * | | | Complain loudly if the main loop is blockedAleksey Kladov2020-01-291-1/+21
| | | | | |
| | * | | | More uniform namingAleksey Kladov2020-01-291-7/+7
| | | | | |
| * | | | | Merge #2933bors[bot]2020-01-291-28/+28
| |\| | | | | | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | 2933: Don't compute diagnostics on the main thread r=matklad a=matklad closes #2909 Co-authored-by: Aleksey Kladov <[email protected]>
| | * | | Don't compute diagnostics on the main threadAleksey Kladov2020-01-291-28/+28
| |/ / / | | | | | | | | | | | | closes #2909
| * | | Merge #2917bors[bot]2020-01-283-14/+88
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2917: Prefer imports starting with std r=matklad a=SomeoneToIgnore Closes https://github.com/rust-analyzer/rust-analyzer/issues/2915 Co-authored-by: Kirill Bulatov <[email protected]>
| | * | | Add the testsKirill Bulatov2020-01-282-0/+42
| | | | |
| | * | | Revert accidental package-lock.json changeKirill Bulatov2020-01-281-1/+1
| | | | |
| | * | | Apply the proposed refactoringKirill Bulatov2020-01-281-38/+43
| | | | |
| | * | | Prefer imports starting with stdKirill Bulatov2020-01-273-7/+34
| | | | |