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

Feature/swift tests #1945

Merged
merged 6 commits into from
Dec 12, 2017
Merged

Feature/swift tests #1945

merged 6 commits into from
Dec 12, 2017

Conversation

idodeclare
Copy link
Contributor

Hello,

Please consider for integration this patch to add Swift tests and to fix the exposed bugs.

Thank you.

Also, this xref demonstrates problems with
handling of numbers and URI collateral capture.
Also:
- Make SwiftXref case-sensitive, converting
  {File} pattern.
- Escape all HTML special characters in
  SwiftXref.
- Fix not to continue STRING after an end-quote,
  in case a TSTRING is actually starting.
- Fix to fine-tune URI link for Swift
  COMMENT,SDOC.
- Fix not to recognize escape sequences in
  TSTRING.
- Fix to eliminate QSTRING which is not valid
  Swift literal syntax.
Also:
- Avoid overlapping spans for comments,strings
  and scopes for readability while debugging.
@tarzanek tarzanek self-assigned this Dec 12, 2017
@tarzanek tarzanek added this to the 1.1 milestone Dec 12, 2017
@tarzanek
Copy link
Contributor

thank you very much for the quote and identifier fix!
merging ...

@tarzanek tarzanek merged commit 07a41d5 into oracle:master Dec 12, 2017
@idodeclare
Copy link
Contributor Author

My pleasure!

@idodeclare idodeclare deleted the feature/swift_tests branch December 13, 2017 01:28
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.

2 participants