Skip to content
This repository has been archived by the owner on Jan 8, 2024. It is now read-only.

20230227 defmac #134

Open
wants to merge 68 commits into
base: base
Choose a base branch
from
Open

20230227 defmac #134

wants to merge 68 commits into from

Conversation

prozacchiwawa
Copy link

Introduces an "advanced macro" system which provides a new form of macros.

(defmac macro-name (X) ...)

is like the existing defmacro system but has important differences:

  • It runs as a fully separate preprocessor phase for chialisp, so it can create top level forms.
  • Its value space includes user focused features like symbols, strings and numbers. unlike the existing chialisp macro system, it can distinguish user inputs to a finer detail than cons or atom.
  • It provides new primitives which preserve user inputs better and allow the user to query and convert user available values.
  • It has access to the defuns and constants from the surrounding program.

@coveralls-official
Copy link

coveralls-official bot commented May 16, 2023

Pull Request Test Coverage Report for Build 5800960865

  • 683 of 816 (83.7%) changed or added relevant lines in 14 files are covered.
  • 2 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.4%) to 78.84%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/compiler/rename.rs 6 7 85.71%
src/compiler/evaluate.rs 61 65 93.85%
src/classic/clvm_tools/cmds.rs 38 43 88.37%
src/compiler/clvm.rs 11 18 61.11%
src/compiler/comptypes.rs 10 23 43.48%
src/compiler/sexp.rs 13 29 44.83%
src/compiler/preprocessor/macros.rs 251 285 88.07%
src/compiler/preprocessor/mod.rs 244 297 82.15%
Files with Coverage Reduction New Missed Lines %
src/classic/clvm_tools/cmds.rs 1 64.24%
src/compiler/evaluate.rs 1 87.15%
Totals Coverage Status
Change from base Build 5799910856: 0.4%
Covered Lines: 7817
Relevant Lines: 9915

💛 - Coveralls

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant