Skip to content

Structured suggestion when not using struct pattern #74211

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 14, 2020
Merged

Structured suggestion when not using struct pattern #74211

merged 1 commit into from
Jul 14, 2020

Conversation

estebank
Copy link
Contributor

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 10, 2020
euclio
euclio reviewed Jul 10, 2020
View reviewed changes
petrochenkov
petrochenkov reviewed Jul 10, 2020
View reviewed changes
@estebank estebank force-pushed the struct-pat-as-unit branch from 07da615 to a4c6841 Compare July 11, 2020 19:16
petrochenkov
petrochenkov reviewed Jul 13, 2020
View reviewed changes
@petrochenkov petrochenkov added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 13, 2020
@estebank estebank added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jul 13, 2020
petrochenkov
petrochenkov reviewed Jul 14, 2020
View reviewed changes
@petrochenkov
Copy link
Contributor

r=me after squashing commits.

@petrochenkov petrochenkov added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 14, 2020
@estebank estebank force-pushed the struct-pat-as-unit branch 2 times, most recently from 3e7d4dc to 692d56f Compare July 14, 2020 16:29
petrochenkov
petrochenkov reviewed Jul 14, 2020
View reviewed changes
@petrochenkov
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Jul 14, 2020

📌 Commit 692d56f30c67b305992047cdfc3674288e189edf has been approved by petrochenkov

@bors
Copy link
Collaborator

bors commented Jul 14, 2020

🌲 The tree is currently closed for pull requests below priority 5, this pull request will be tested once the tree is reopened

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jul 14, 2020
@estebank
Suggest struct pat on incorrect unit or tuple pat
0429820 
When encountering a unit or tuple pattern for a struct-like item, suggest
using the correct pattern.

Use `insert_field_names_local` when evaluating variants and store field
names even when the list is empty in order to produce accurate
structured suggestions.
@estebank estebank force-pushed the struct-pat-as-unit branch from 692d56f to 0429820 Compare July 14, 2020 17:41
@estebank
Copy link
Contributor Author

@bors r=petrochenkov

@bors
Copy link
Collaborator

bors commented Jul 14, 2020

📌 Commit 0429820 has been approved by petrochenkov

@bors
Copy link
Collaborator

bors commented Jul 14, 2020

🌲 The tree is currently closed for pull requests below priority 5, this pull request will be tested once the tree is reopened

@Manishearth Manishearth mentioned this pull request Jul 14, 2020
Merged
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 14, 2020
@bors
Auto merge of rust-lang#74342 - Manishearth:rollup-l63pesj, r=Manishe…
23744c8 
…arth

Rollup of 11 pull requests

Successful merges:

 - rust-lang#73759 (Add missing Stdin and StdinLock examples)
 - rust-lang#74211 (Structured suggestion when not using struct pattern)
 - rust-lang#74228 (Provide structured suggestion on unsized fields and fn params)
 - rust-lang#74252 (Don't allow `DESTDIR` to influence LLVM builds)
 - rust-lang#74263 (Slight reorganization of sys/(fast_)thread_local)
 - rust-lang#74271 (process_unix: prefer i32::*_be_bytes over manually shifting bytes)
 - rust-lang#74272 (pprust: support multiline comments within lines)
 - rust-lang#74332 (Update cargo)
 - rust-lang#74334 (bootstrap: Improve wording on docs for `verbose-tests`)
 - rust-lang#74336 (typeck: use `item_name` in cross-crate packed diag)
 - rust-lang#74340 (lint: use `transparent_newtype_field` to avoid ICE)

Failed merges:

r? @ghost
@bors
Copy link
Collaborator

bors commented Jul 14, 2020

☔ The latest upstream changes (presumably #74342) made this pull request unmergeable. Please resolve the merge conflicts.

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jul 14, 2020
@bors bors merged commit be5c7ab into rust-lang:master Jul 14, 2020
@petrochenkov petrochenkov mentioned this pull request Jul 15, 2020
Merged
@petrochenkov petrochenkov mentioned this pull request Aug 5, 2020
Closed
@estebank estebank mentioned this pull request Aug 10, 2020
Merged
@cuviper cuviper added this to the 1.47.0 milestone May 2, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@euclio euclio euclio left review comments

@petrochenkov petrochenkov petrochenkov left review comments

Assignees

@petrochenkov petrochenkov

Labels
S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author.
Projects
None yet
Milestone
1.47.0
Development

Successfully merging this pull request may close these issues.
6 participants
@estebank @petrochenkov @bors @euclio @cuviper @rust-highfive