Skip to content

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
Contributor

@swift-ci Please smoke test

@slavapestov
Copy link
Contributor

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
Contributor

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.

4 participants