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

[DiagnosticsQoI] SR-3599: Better recovery for decls with consecutive identifiers #6948

Merged

Conversation

matthewcarroll
Copy link
Contributor

Add diagnostics to fix decls with consecutive identifiers. This applies to
types, properties, variables, and enum cases. The diagnostic adds a camel-cased option if it is different than the first option.

let a b = 0
// error: found an unexpected second identifier in variable declaration; is there an accidental break?
// note: join the identifiers together
  var a b = 0
      ~~^
      ab

https://bugs.swift.org/browse/SR-3599

…identifiers

Add diagnostics to fix decls with consecutive identifiers. This applies to
types, properties, variables, and enum cases. The diagnostic adds a camel-cased option if it is different than the first option.

https://bugs.swift.org/browse/SR-3599
@slavapestov
Copy link
Member

@swift-ci Please smoke test

@slavapestov
Copy link
Member

Looks like you accidentally fixed a compiler_crasher.

@matthewcarroll
Copy link
Contributor Author

@slavapestov,

Would you like me to remove the expectation of crashing in the test 28619-basety-islvaluetype-basety-is-anymetatypetype?

@CodaFi
Copy link
Member

CodaFi commented Jan 22, 2017

@matthewcarroll If you want to automate the process,

../llvm/utils/lit/lit.py -sv ../build/Ninja-DebugAssert/swift-macosx-x86_64/validation-test-macosx-x86_64/compiler_crashers/ | ./utils/resolve-crashes.py

…mpiler crashers fixed test suite.

The crash was fixed accidentally in 93a7a9f.
@matthewcarroll
Copy link
Contributor Author

@slavapestov, @CodaFi,

Thanks! I ran the script on the test.

@DougGregor
Copy link
Member

@swift-ci please smoke test

@DougGregor DougGregor self-assigned this Jan 23, 2017
@DougGregor
Copy link
Member

@swift-ci please smoke test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants