Skip to content
This repository was archived by the owner on May 20, 2026. It is now read-only.

Fix #37#39

Open
sorawee wants to merge 1 commit into
racket:masterfrom
sorawee:fix-37
Open

Fix #37#39
sorawee wants to merge 1 commit into
racket:masterfrom
sorawee:fix-37

Conversation

@sorawee
Copy link
Copy Markdown
Contributor

@sorawee sorawee commented Jun 9, 2022

Parse 'collection and 'implies when we retrieve the information.
This is done in a way that is as compliant with the existing format
as much as possible for valid values. However, invalid values will be
explicitly marked / quoted so that they cause no trouble.

Two significant changes for valid values:

  • a 'core in 'implies will be further wrapped in a list to distinguish
    a symbol from a string.
  • a 'use-pkg-name will be further wrapped in a list to distinguish
    a symbol from a string.

These two break the compliance, but originally it's not possible to
unambiguously use this information anyway.

Parse 'collection and 'implies when we retrieve the information.
This is done in a way that is as compliant with the existing format
as much as possible for valid values. However, invalid values will be
explicitly marked / quoted so that they cause no trouble.

Two significant changes for valid values:

- a 'core in 'implies will be further wrapped in a list to distinguish
  a symbol from a string.
- a 'use-pkg-name will be further wrapped in a list to distinguish
  a symbol from a string.

These two break the compliance, but originally it's not possible to
unambiguously use this information anyway.
@sorawee
Copy link
Copy Markdown
Contributor Author

sorawee commented Jun 9, 2022

Please don't merge this yet. I meant this to be a PR for discussion.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant