Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add "Use of uninitialized variable" diagnostic #3315

Merged
merged 3 commits into from
Feb 11, 2019

Conversation

artemmukhin
Copy link
Member

@artemmukhin artemmukhin commented Jan 31, 2019

Fixes #3162

Merge after #3320

screenshot 2019-01-31 at 18 44 28

@mchernyavsky mchernyavsky self-assigned this Feb 1, 2019
@mchernyavsky mchernyavsky changed the title Add "Use of uninitialized variable" diagnostic [WIP] Add "Use of uninitialized variable" diagnostic Feb 10, 2019
@artemmukhin artemmukhin changed the title [WIP] Add "Use of uninitialized variable" diagnostic Add "Use of uninitialized variable" diagnostic Feb 11, 2019
Copy link
Member

@mchernyavsky mchernyavsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

bors bot added a commit that referenced this pull request Feb 11, 2019
3315: Add "Use of uninitialized variable" diagnostic r=mchernyavsky a=ortem

Fixes #3162

Merge after #3320 

<img width="340" alt="screenshot 2019-01-31 at 18 44 28" src="https://user-images.githubusercontent.com/4854600/52065825-5e563a00-2588-11e9-8de1-859e0fa3504c.png">


Co-authored-by: ortem <ortem00@gmail.com>
@bors
Copy link
Contributor

bors bot commented Feb 11, 2019

@bors bors bot merged commit 9843188 into intellij-rust:master Feb 11, 2019
bors bot added a commit that referenced this pull request Mar 9, 2019
3341: Add "Initialize with a default value" quick fix r=vlad20012 a=ortem

Add the quick fix for "Use of uninitialized variable" error. Includes the refactoring of `AddStructFieldsFix.kt`.
Merge after #3315 

![1](https://user-images.githubusercontent.com/4854600/52709534-137fed80-2f9e-11e9-928a-7363cc84bdda.gif)


Co-authored-by: ortem <ortem00@gmail.com>
@farodin91 farodin91 mentioned this pull request Sep 4, 2020
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants