mattwparas / steel
51%

DEFAULT BRANCH: master
Repo Added
Files 124
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH master
branch: SELECT
CHANGE BRANCH
x

coverage: 50.756% (+0.3%) from 50.485%
22936788704

push

github

web-flow 
make the port procedures respect `parameterize` (#644)

* allow defaulting the port for (read-port-to-<string/bytes>)

and adjust the docs for that

* make parameterizing work on read-char, read-byte, etc

trying to parameterize the read-char / read-byte / etc procedures
previously didn't work, as they were implemented in rust, where the
rust builtin would just fall back to stdin / stdout for default ports.

the easiest way to make parameterizing work is by making the rust
builtins hidden and basically writing wrapper procedures on the steel
side, where parameterizing already just works.

this also means, that the rust builtins can now just take two arguments
instead of trying to make it callable with only one, eliminating the
need for the `io_args` procedures.

before:

```
λ > (define port (open-output-string))
λ > (parameterize ([current-output-port port])
      (write-char #\a))
a
λ > (get-output-string port)
=> ""
```

after:

```
λ > (define port (open-output-string))
λ > (parameterize ([current-output-port port])
      (write-char #\a))
λ > (get-output-string port)
=> "a"
```

* copy port docs to scheme module

* cargo xtask docgen

* rename default-current-<direction>-port to just default-<direction>-port

those procedures were previously also used in the read-port etc
procedures, but since those have been moved to the steel side to fix
parameterizing, these are no longer needed as the rust-side private
builtins can just take a port unconditionally.

in my opinion calling them something with "current" is a little
misleading, as these are just constructors for the default error ports,
as they are not affected by parameterizing, which the
current-<direction>-port should be. additionally it the functions better
up with the private builtin wrapper procedures (their only call site),
as those are just called `#%default-<direction>-port`.

* add some tests for parameterizing

18 of 18 new or added lines in 2 files covered. (100.0%)

81 existing lines in 7 files now uncovered.

19714 of 38841 relevant lines covered (50.76%)

3611280.18 hits per line

Relevant lines Covered
38841 RELEVANT LINES 19714 COVERED LINES
3611280.18 HITS PER LINE
Source Files on master

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
22936788704 master make the port procedures respect `parameterize` (#644) * allow defaulting the port for (read-port-to-<string/bytes>) and adjust the docs for that * make parameterizing work on read-char, read-byte, etc trying to parameterize the read-char / re... push web-flow github
50.76
22936143927 master Missing subtract impl in JIT (#647) * add impl for subtraction to handle slow path * fix missing number case push web-flow github
50.49
22932797198 mwp-missing-jit-impl-subtract Merge e928c03e2 into ee19ab5e5 Pull #647 web-flow github
50.48
22930017765 master don't eagerly expand top level macros (#646) push web-flow github
50.81
22927621916 mwp-fix-repl-macros-leaking-top-level Merge e418d63f8 into 6a0de2453 Pull #646 web-flow github
50.82
22879885865 parameterize-builtin-port-procedures Merge c7968edbe into 6a0de2453 Pull #644 web-flow github
50.49
22826422057 master Feat: initial implementation of here string (#641) * here string: initial implementation * here string: initial implementation * here string: tests and formatting * here string: docs push web-flow github
50.82
22824858619 feature-here-doc Merge 903fbef9d into 0d51199cd Pull #641 web-flow github
50.83
22814645764 mwp-starting-more-specialization-of-primitives Merge ba9e75d21 into 0d51199cd Pull #643 web-flow github
51.04
22785775448 feature-here-doc Merge a320bc6be into a20c1a54d Pull #641 web-flow github
50.8
See All Builds (2006)