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

Clarify where table 4 applies #975

Open
kmiller68 opened this Issue Aug 16, 2017 · 1 comment

Comments

Projects
None yet
2 participants
@kmiller68

kmiller68 commented Aug 16, 2017

Reading the spec it's very unclear to me where table 4 applies. In fact I find it actively misleading. In sections 18-26 + Annex B.2 table 4 no longer applies since it is overridden by the last couple paragraphs of Section 17. In fact, I can't find any place in the spec where it applies. Although, I may have missed something.

Does it make sense to make an editorial change and move the last couple paragraphs of Section 17 to where table 4 is now (and delete table 4 assuming it doesn't apply anywhere)?

@littledan

This comment has been minimized.

Show comment
Hide comment
@littledan

littledan Aug 17, 2017

Member

Isn't it used by ToPropertyDescriptor?

If this is the only place where it's used, I agree that it would be fine to move that logic into ToPropertyDescriptor, and instead say where Table 4 appears, "either this set of fields must exist, or that set".

Member

littledan commented Aug 17, 2017

Isn't it used by ToPropertyDescriptor?

If this is the only place where it's used, I agree that it would be fine to move that logic into ToPropertyDescriptor, and instead say where Table 4 appears, "either this set of fields must exist, or that set".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment