Skip to content

Follow-up on the bridging rework #69071

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Oct 9, 2023
Merged

Follow-up on the bridging rework #69071

merged 2 commits into from
Oct 9, 2023

Conversation

@eeckstein
Copy link
Contributor

@eeckstein eeckstein commented Oct 9, 2023 •
edited
Loading

This is a follow-up on #69039

  • fixing an uninitialized member variable in BridgedSourceLoc
  • some refactoring to simplify the code in BridgedBuilder

@eeckstein
Copy link
Contributor Author

eeckstein commented Oct 9, 2023

@swift-ci smoke test and merge

@xedin xedin removed their request for review October 9, 2023 17:18
@swift-ci swift-ci merged commit 4430799 into swiftlang:main Oct 9, 2023
@eeckstein eeckstein deleted the bridging-followup branch October 9, 2023 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hborla hborla Awaiting requested review from hborla hborla is a code owner

@slavapestov slavapestov Awaiting requested review from slavapestov slavapestov is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@eeckstein @swift-ci