aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Merge #2249bors[bot]2019-11-151-111/+93
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2249: Cleanup hover r=matklad a=kjeremy Take advantage of classify_name to consolidate multiple hover paths. This isn't quite as clean as I want it to be (`no_fallback` bool is wonky). There's a relationship between `HoverResult` being empty and the range that is a little warty. Also I noticed that HoverResults are always marked as exact and have been for quite a while... maybe that should be removed in another PR. Co-authored-by: Jeremy Kolb <[email protected]>
| * | | Cleanup hoverJeremy Kolb2019-11-151-111/+93
| | |/ | |/| | | | | | | Take advantage of classify_name
* | | Merge #2246bors[bot]2019-11-152-20/+20
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | 2246: Normalize Line Endings r=matklad a=kjeremy Line endings with git always confuse me so let's be explicit. In doing this I ran `git add --renormalize .` on my repo per https://help.github.com/en/github/using-git/configuring-git-to-handle-line-endings and it converted a bunch of files. I then backed it all out and ran it again and only this one `Cargo.toml` file changed so I suspect line endings are just weird. Co-authored-by: kjeremy <[email protected]>
| * | Renormalize line endingskjeremy2019-11-151-19/+19
| | |
| * | Set text to autodetect and use LFkjeremy2019-11-151-0/+1
| | |
| * | Remove snapshots since we no long commit themkjeremy2019-11-151-1/+0
|/ /
* | Merge #2245bors[bot]2019-11-143-20/+22
|\ \ | | | | | | | | | | | | | | | | | | | | | 2245: Even if jemalloc feature is used do not use it on msvc r=matklad a=kjeremy Fixes #2233 Co-authored-by: kjeremy <[email protected]>
| * | Even if jemalloc feature is used do not use it on msvckjeremy2019-11-143-20/+22
|/ / | | | | | | Fixes #2233
* | Merge #2243bors[bot]2019-11-1414-94/+129
|\ \ | | | | | | | | | | | | | | | | | | | | | 2243: Move body queries to hir_def r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Move body queries to hir_defAleksey Kladov2019-11-1412-75/+110
| | |
| * | Simplify parsingAleksey Kladov2019-11-142-19/+19
|/ /
* | Merge #2240bors[bot]2019-11-147-368/+360
|\ \ | | | | | | | | | | | | | | | | | | | | | 2240: Move scopes to hir_def r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Move scopes to hir_defAleksey Kladov2019-11-147-368/+360
| | |
* | | Merge #2238bors[bot]2019-11-142-10/+7
|\| | | | | | | | | | | | | | | | | | | | | | | 2238: Slightly better naming r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Slightly better namingAleksey Kladov2019-11-142-10/+7
|/ /
* | Merge #2237bors[bot]2019-11-143-13/+25
|\ \ | | | | | | | | | | | | | | | | | | | | | 2237: Revert #2230 r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Revert #2230Aleksey Kladov2019-11-143-13/+25
|/ / | | | | | | | | Looks like autocfg tries to do slightly more than we need (see #2231), so let's stick with minimal home-grown solution.
* | Merge #2235bors[bot]2019-11-147-100/+156
|\ \ | | | | | | | | | | | | | | | | | | | | | 2235: Handle macro-generated expressions slightly less wrong r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Handle macro-generated expressions slightly less wrongAleksey Kladov2019-11-146-48/+62
| | |
| * | Move expansion to ExpanderAleksey Kladov2019-11-142-27/+41
| | |
| * | Move parse_path to ExpanderAleksey Kladov2019-11-142-10/+10
| | |
| * | Expansion stack scaffoldAleksey Kladov2019-11-142-8/+38
| | |
| * | Add Expader::to_sourceAleksey Kladov2019-11-142-10/+8
| | |
| * | Move original_file to ExpanderAleksey Kladov2019-11-142-9/+11
| | |
| * | Rename MacroResolver -> ExpanderAleksey Kladov2019-11-143-29/+25
| | |
| * | Move current file to MacroResolverAleksey Kladov2019-11-143-21/+27
| | |
| * | Reduce visibilityAleksey Kladov2019-11-141-5/+1
| | |
* | | Merge #2234bors[bot]2019-11-141-14/+9
|\| | | | | | | | | | | | | | | | | | | | | | | 2234: Normalize data r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Normalize dataAleksey Kladov2019-11-141-14/+9
|/ / | | | | | | No need to store derivable info
* | Merge #2232bors[bot]2019-11-137-29/+36
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2232: Use anyhow::Result in xtask, add contexts r=matklad a=killercup This builds on #2231 but was actually done before that. You see, the cause for #2231 was that I got this error message: Error: Error { kind: Io(Os { code: 2, kind: NotFound, message: "No such file or directory" }) } Just switching to `anyhow::Result` got me stack traces (when setting `RUST_LIB_BACKTRACE=1`) that at least showed stack backtrace: 0: std::backtrace::Backtrace::create 1: std::backtrace::Backtrace::capture 2: anyhow::error::<impl core::convert::From<E> for anyhow::Error>::from 3: xtask::install_server 4: xtask::install 5: xtask::main 6: std::rt::lang_start::{{closure}} 7: std::panicking::try::do_call 8: __rust_maybe_catch_panic 9: std::rt::lang_start_internal 10: std::rt::lang_start 11: main With the added contexts (not at all exhaustive), the error became Error: install server Caused by: 0: build AutoCfg with target directory 1: No such file or directory (os error 2) Since anyhow is such a small thing (no new transitive dependencies!), and in general gives you `Result<T, Box<dyn Error>>` on steroids, I think this a nice small change. The only slightly annoying thing was to replace all the `Err(format!(…))?` calls (haven't even looked at whether we can make it support wrapping strings though), but the `bail!` macro is shorter anyway :) Co-authored-by: Pascal Hertleif <[email protected]>
| * | Use anyhow::Result in xtask, add contextsPascal Hertleif2019-11-137-29/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This builds on #2231 but was actually done before that. You see, the cause for #2231 was that I got this error message: Error: Error { kind: Io(Os { code: 2, kind: NotFound, message: "No such file or directory" }) } Just switching to `anyhow::Result` got me stack traces (when setting `RUST_LIB_BACKTRACE=1`) that at least showed stack backtrace: 0: std::backtrace::Backtrace::create 1: std::backtrace::Backtrace::capture 2: anyhow::error::<impl core::convert::From<E> for anyhow::Error>::from 3: xtask::install_server 4: xtask::install 5: xtask::main 6: std::rt::lang_start::{{closure}} 7: std::panicking::try::do_call 8: __rust_maybe_catch_panic 9: std::rt::lang_start_internal 10: std::rt::lang_start 11: main With the added contexts (not at all exhaustive), the error became Error: install server Caused by: 0: build AutoCfg with target directory 1: No such file or directory (os error 2) Since anyhow is such a small thing (no new transitive dependencies!), and in general gives you `Result<T, Box<dyn Error>>` on steroids, I think this a nice small change. The only slightly annoying thing was to replace all the `Err(format!(…))?` calls (haven't even looked at whether we can make it support wrapping strings though), but the `bail!` macro is shorter anyway :)
* | | Merge #2231bors[bot]2019-11-131-1/+2
|\| | | | | | | | | | | | | | | | | | | | | | | 2231: Allow usage of CARGO_TARGET_DIR env var r=matklad a=killercup Co-authored-by: Pascal Hertleif <[email protected]>
| * | Allow usage of CARGO_TARGET_DIR env varPascal Hertleif2019-11-131-1/+2
|/ /
* | Merge #2230bors[bot]2019-11-133-25/+12
|\ \ | | | | | | | | | | | | | | | | | | | | | 2230: Use autocfg to determine rust version r=matklad a=kjeremy Fixes #2229 Co-authored-by: kjeremy <[email protected]>
| * | Use autocfg to determine rust versionkjeremy2019-11-133-25/+12
|/ /
* | Merge #2228bors[bot]2019-11-131-30/+30
|\ \ | | | | | | | | | | | | | | | | | | | | | 2228: Update Crates r=matklad a=kjeremy Co-authored-by: Jeremy Kolb <[email protected]>
| * | Update CratesJeremy Kolb2019-11-131-30/+30
|/ /
* | Merge #2226bors[bot]2019-11-134-71/+132
|\ \ | | | | | | | | | | | | | | | | | | | | | 2226: Use strongly-typed ast building for early-return assist r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Make make:: builders slightly more convenientAleksey Kladov2019-11-132-9/+12
| | |
| * | Use strongly-typed ast building for early-return assistAleksey Kladov2019-11-132-45/+102
| | |
| * | Add a bit of typesAleksey Kladov2019-11-131-5/+5
| | |
| * | Minor cleanupAleksey Kladov2019-11-132-20/+21
| | |
* | | Merge #2225bors[bot]2019-11-137-70/+55
|\| | | | | | | | | | | | | | | | | | | | | | | 2225: Reduce duplication between uncertain floats & ints r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Reduce duplication between uncertain floats & intsAleksey Kladov2019-11-137-70/+55
|/ /
* | Merge #2223bors[bot]2019-11-128-732/+765
|\ \ | | | | | | | | | | | | | | | | | | | | | 2223: Move expression lowering to hir_def r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | Add helpful pointer to module docsAleksey Kladov2019-11-121-0/+2
| | |
| * | Drop obsolete commentAleksey Kladov2019-11-121-4/+0
| | |
| * | Move expression lowering to hir_defAleksey Kladov2019-11-127-732/+767
| | |
* | | Merge #2217bors[bot]2019-11-123-11/+23
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | 2217: Implement FromStr for enum Edition r=matklad a=clemarescx Just did this as I came across the comment in the code asking for implementing `std::str::FromStr` for `input::Edition`. Not sure what was meant by "proper error handling" though, `panic!` with a descriptive message might not be it :sweat_smile: Co-authored-by: Metabaron <[email protected]>
| * | Fix unused importMetabaron2019-11-121-1/+1
| | |