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

Explicitly mark objects of structured type with base class data #1

Closed
wants to merge 1 commit into from
Closed

Explicitly mark objects of structured type with base class data #1

wants to merge 1 commit into from

Conversation

egranata
Copy link
Contributor

Base class data is vended as "just another child", which is mostly OK, except sometimes it's useful to know when the base class is being parsed vs. a member variable
Relying on the name "super" isn't actually correct, but at the Mirror level we actually know base class vs. ivar

This change propagates that knowledge via adding an extra byte to the payload for structured data in the logger format, such that the logger can tell consumers whether the 0th child is the base class or the first member variable

rdar://problem/19072746

Base class data is vended as "just another child", which is mostly OK, except sometimes it's useful to know when the base class is being parsed vs. a member variable
Relying on the name "super" isn't actually correct, but at the Mirror level we actually know base class vs. ivar

This change propagates that knowledge via adding an extra byte to the payload for structured data in the logger format, such that the logger can tell consumers whether the 0th child is the base class or the first member variable

rdar://problem/19072746
@shahmishal shahmishal closed this Oct 6, 2020
@shahmishal
Copy link
Member

The Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please re-create the pull request with main branch.

More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412

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

2 participants