Document and test std/sugar, std/source, plus fixes #993
Conversation
✅ Deploy Preview for elastic-ritchie-8f47f9 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
doc/reference/std/sugar.md
Outdated
| ``` Scheme | ||
| > (import :std/error) | ||
| > (def (unbound-runtime-exception? e) | ||
| (match e ((RuntimeException exception: (? unbound-global-exception?)) #t) |
There was a problem hiding this comment.
just unbound-global-exception? Does this for you.
There was a problem hiding this comment.
Not sure what you mean...
There was a problem hiding this comment.
the runtime already wraps the gambit exception predicates to unwrap them from RuntimeException.
| @@ -97,27 +171,88 @@ you thus have to pass the receiver as first argument to the method. | |||
| The advantage over `using` is that there is no implicit allocation for | |||
There was a problem hiding this comment.
stale, there is no advantage over using.
There was a problem hiding this comment.
Should I remove altogether from sugar? What exactly do I remove or keep?
There was a problem hiding this comment.
there used to be an older using macro, got removed with the intro of real using.
There was a problem hiding this comment.
At this point, I'll let this PR get in, and you can remove the code and docs and tests.
No description provided.