Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Turn Ty::Tuple variant into a tuple-variant | Lukas Wirth | 2021-02-28 | 8 | -22/+20 |
| | |||||
* | Merge #7813 | bors[bot] | 2021-02-28 | 17 | -943/+709 |
|\ | | | | | | | | | | | | | | | 7813: Inline TypeCtor into Ty r=flodiebold a=Veykril This removes the `ApplicationTy` variant from `Ty` bringing the representation a lot closer to chalk's `TyKind`. Co-authored-by: Lukas Wirth <[email protected]> | ||||
| * | Fix code_model::Type::walk not walking all types | Lukas Wirth | 2021-02-28 | 3 | -26/+25 |
| | | |||||
| * | Inline TypeCtor into Ty | Lukas Wirth | 2021-02-28 | 17 | -936/+703 |
| | | |||||
* | | Use consistent naming for assist | Aleksey Kladov | 2021-02-28 | 4 | -39/+39 |
| | | |||||
* | | Don't spam loop-rewriting assist | Aleksey Kladov | 2021-02-28 | 2 | -2/+22 |
| | | | | | | | | The more focused the assist, the better! | ||||
* | | Merge #7804 | bors[bot] | 2021-02-28 | 13 | -419/+270 |
|\| | | | | | | | | | | | | | | | 7804: Introduce TypeCtor::Scalar r=lnicola a=Veykril `TypeCtor::Int(..) | TypeCtor::Float(..) | TypeCtor::Char | TypeCtor::Bool` => `TypeCtor::Scalar(..)`, in this case we can actually just straight up use `chalk_ir::Scalar` already since its just a POD without any IDs or anything. Co-authored-by: Lukas Wirth <[email protected]> | ||||
| * | Use chalk_ir::Scalar directly | Lukas Wirth | 2021-02-28 | 6 | -197/+78 |
| | | |||||
| * | Introduce TypeCtor::Scalar | Lukas Wirth | 2021-02-28 | 13 | -380/+350 |
| | | |||||
* | | Merge #7808 | bors[bot] | 2021-02-28 | 1 | -17/+2 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | 7808: Fix assert split exprs on comma r=edwin0cheng a=edwin0cheng bors r+ Co-authored-by: Edwin Cheng <[email protected]> | ||||
| * | | Fix assert split exprs on comma | Edwin Cheng | 2021-02-28 | 1 | -17/+2 |
| | | | |||||
* | | | Fixed remaining references to `AnalysisChange` (now: `Change`) | Vincent Esche | 2021-02-28 | 3 | -3/+3 |
| | | | | | | | | | | | | (The type was renamed/moved in 8716c4cec3a05ba891b20b5f28df69d925b913ad) | ||||
* | | | For unresolved macros, hightlight only the last segment | Aleksey Kladov | 2021-02-28 | 8 | -109/+209 |
|/ / | |||||
* | | Fix non-latin characters doc comment for mbe | Edwin Cheng | 2021-02-28 | 2 | -1/+24 |
| | | |||||
* | | Merge #7802 | bors[bot] | 2021-02-28 | 5 | -89/+144 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7802: Fix builtin macros split exprs on comma r=edwin0cheng a=edwin0cheng Fixes #7640 bors r+ Co-authored-by: Edwin Cheng <[email protected]> | ||||
| * | | Fix builtin macros split exprs on comma | Edwin Cheng | 2021-02-28 | 5 | -89/+144 |
| | | | |||||
* | | | Restrict visibilities to the containing DefMap | Jonas Schievink | 2021-02-28 | 2 | -4/+46 |
|/ / | |||||
* | | Merge #7797 | bors[bot] | 2021-02-27 | 1 | -2/+6377 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7797: Format generated lints and features manually r=matklad a=lnicola As `quote` and `rustfmt` leave them on a single line, which makes running `grep` in the repository quite annoying. Also removes a dead `gen_features.rs` file (`gen_lint_completions.rs` does the same thing). Co-authored-by: Laurențiu Nicola <[email protected]> | ||||
| * | | Format generated features manually instead of relying on rustfmt | Laurențiu Nicola | 2021-02-27 | 1 | -2/+6377 |
| | | | |||||
* | | | Merge #7566 | bors[bot] | 2021-02-27 | 4 | -0/+222 |
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7566: Add benchmark tests for mbe r=matklad a=edwin0cheng This PR add more real world tests dumped from `rust-analyzer analysis-stats .` to benchmark its performance. cc #7513 r? @matklad Co-authored-by: Edwin Cheng <[email protected]> | ||||
| * | | | Add benchmark test for mbe | Edwin Cheng | 2021-02-24 | 4 | -0/+222 |
| | | | | |||||
* | | | | generate try_into instead of into | Domantas Jadenkus | 2021-02-27 | 3 | -72/+96 |
| | | | | |||||
* | | | | rename generate_enum_match_method file to match assist name | Domantas Jadenkus | 2021-02-27 | 2 | -2/+2 |
| | | | | |||||
* | | | | move into_ and as_ generation to a separate file | Domantas Jadenkus | 2021-02-27 | 4 | -402/+340 |
| | | | | |||||
* | | | | cargo fmt | Domantas Jadenkus | 2021-02-27 | 1 | -35/+20 |
| | | | | |||||
* | | | | add generate_enum_as_method assist | Domantas Jadenkus | 2021-02-27 | 3 | -0/+154 |
| | | | | |||||
* | | | | deduplicate some | Domantas Jadenkus | 2021-02-27 | 1 | -37/+28 |
| | | | | |||||
* | | | | add generate_enum_into_method assist | Domantas Jadenkus | 2021-02-27 | 3 | -20/+275 |
| | | | | |||||
* | | | | make generate_enum_is_method work on any variants | Domantas Jadenkus | 2021-02-27 | 1 | -18/+95 |
| | | | | |||||
* | | | | rename existing assist to generate_enum_is_method | Domantas Jadenkus | 2021-02-27 | 3 | -11/+11 |
| |_|/ |/| | | |||||
* | | | Disable "Flip comma" assist inside a macro call | Denys Zadorozhnyi | 2021-02-26 | 1 | -2/+24 |
| | | | |||||
* | | | docs(ide_assists): Change importMergeBehaviour to importMergeBehavior | Sanchith Hegde | 2021-02-26 | 1 | -3/+3 |
| | | | |||||
* | | | bump crates | kjeremy | 2021-02-25 | 1 | -1/+1 |
| | | | |||||
* | | | Merge #7741 | bors[bot] | 2021-02-24 | 4 | -0/+327 |
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7741: Add convert_for_to_iter_for_each assist r=mattyhall a=mattyhall Implements one direction of #7681 I wonder if this tries to guess too much at the right thing here. A common pattern is: ```rust let col = vec![1, 2, 3]; for v in &mut col { *v *= 2; } // equivalent to: col.iter_mut().for_each(|v| *v *= 2); ``` I've tried to detect this case by checking if the expression after the `in` is a (mutable) reference and if not inserting iter()/iter_mut(). This is just a convention used in the stdlib however, so could sometimes be wrong. I'd be happy to make an improvement for this, but not sure what would be best. A few options spring to mind: 1. Only allow this for types that are known to have iter/iter_mut (ie stdlib types) 2. Try to check if iter/iter_mut exists and they return the right iterator type 3. Don't try to do this and just add `.into_iter()` to whatever is after `in` Co-authored-by: Matt Hall <[email protected]> | ||||
| * | | Address further review comments | Matt Hall | 2021-02-24 | 2 | -51/+36 |
| | | | | | | | | | | | | | | | | | | * Use known names for iter/iter_mut method (simplifies checking if the method exists * Extract code to check assist with fixtures to function | ||||
| * | | Address review comments | Matt Hall | 2021-02-23 | 1 | -33/+125 |
| | | | | | | | | | | | | | | | * Move code to build replacement into closure * Look for iter/iter_mut methods on types behind reference | ||||
| * | | Add convert_for_to_iter_for_each assist | Matt Hall | 2021-02-23 | 3 | -0/+250 |
| |/ | |||||
* | | Add tests for apply_demorgan | Lukas Wirth | 2021-02-24 | 3 | -18/+83 |
| | | |||||
* | | De Morgan's Law assist now correctly inverts <, <=, >, >=. | lbrande | 2021-02-24 | 7 | -13/+68 |
| | | |||||
* | | De Morgan's Law assist now correctly parenthesizes binary expressions. | lbrande | 2021-02-24 | 3 | -11/+7 |
| | | |||||
* | | Pickup ConstReference patterns in FindUsages | Lukas Wirth | 2021-02-23 | 2 | -4/+36 |
|/ | |||||
* | is_visible_from_def_map: handle block expressions | Jonas Schievink | 2021-02-23 | 3 | -9/+37 |
| | |||||
* | Add test | Jonas Schievink | 2021-02-23 | 1 | -0/+28 |
| | |||||
* | 7526: Renamed create ssr to ide_ssr. | Chetan Khilosiya | 2021-02-22 | 15 | -8/+9 |
| | |||||
* | 7526: Rename crate assists to ide_assists. | Chetan Khilosiya | 2021-02-22 | 71 | -6/+6 |
| | |||||
* | Add isize to the list of suffixed integers in typed_integer | Jonas Bushart | 2021-02-22 | 1 | -2/+1 |
| | | | | | | The missing `isize` in `typed_integers` seems to just be an oversight. Might fix: #7751 | ||||
* | Make more common assist easier to ues | Aleksey Kladov | 2021-02-22 | 4 | -42/+87 |
| | |||||
* | Add test from #7273 | Laurențiu Nicola | 2021-02-21 | 1 | -0/+45 |
| | |||||
* | Bump deps | Laurențiu Nicola | 2021-02-21 | 2 | -4/+4 |
| | |||||
* | Stop mixing Result and Option with ? in inline_local_variable | Scott McMurray | 2021-02-20 | 1 | -4/+4 |
| | | | | Depending on the discussion in RFC#3058 this might not end up being necessary, but I think it's a reasonable change regardless. |