Skip to content

Comments

Rewrite find-meta function to support r7rs#16

Merged
ufo5260987423 merged 2 commits intoMoganLab:mainfrom
VSteveHL:main
Jul 2, 2025
Merged

Rewrite find-meta function to support r7rs#16
ufo5260987423 merged 2 commits intoMoganLab:mainfrom
VSteveHL:main

Conversation

@VSteveHL
Copy link
Collaborator

@VSteveHL VSteveHL commented Jul 2, 2025

  • Refactored the find-meta function in analysis/identifier/meta.sls using case-lambda, adding the top-environment parameter to distinguish between r6rs and r7rs.
  • Added symbol information for r7rs in analysis/identifier/meta.sls.

- Refactored the `find-meta` function in `analysis/identifier/meta.sls` using `case-lambda`, adding the `top-environment` parameter to distinguish between `r6rs` and `r7rs`.
- Added symbol information for `r7rs` in `analysis/identifier/meta.sls`.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

line 33: redundant

- Unified the naming of R7RS symbol information to a format like `(scheme base)`.
- Removed several leftover debug `print` statements.
@VSteveHL
Copy link
Collaborator Author

VSteveHL commented Jul 2, 2025

I have renamed the R7RS symbol information in analysis/identifier/meta.sls to follow the format (scheme base). Based on my research, this naming convention is consistent with how R7RS libraries are typically introduced. Here are some references that support this approach:

  • Goldfish Scheme: The README includes a section on R7RS Standard Libraries. Link
  • S7 Scheme: The r7rs.scm source file. Link
  • Racket-R7RS: A third-party library for Racket implementing the R7RS-small standard. Link

Additionally, I reviewed the existing symbol information in analysis/identifier/meta.sls and confirmed that this naming convention does not conflict with previously defined symbols. Given this, I believe using the (scheme base) format for R7RS symbol information is a reasonable and consistent choice.

@ufo5260987423 ufo5260987423 merged commit 514d1f3 into MoganLab:main Jul 2, 2025
1 check passed
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