search

aconchillo / guile-json

JSON module for Guile
GNU General Public License v3.0
100 stars 34 forks source link

* json/builder.scm: Validate the scheme object using a function calle… #43

Closed rain-1 closed 4 years ago

rain-1 commented 5 years ago

…d json-valid? before printing partial contents to the output port.

Fixes https://github.com/aconchillo/guile-json/issues/36

aconchillo commented 5 years ago

I have a local branch that is very similar to what you are trying to do. I haven't had time to finish it, but I will do soon.

rain-1 commented 5 years ago

whats wrong with this version?

aconchillo commented 5 years ago

Oh, my bad! I'm so sorry. I didn't look at changes properly and thought it was unfinished. I will take a look and merge asap. Sorry again!

aconchillo commented 5 years ago

It would be great to add some tests in tests.

aconchillo commented 4 years ago

superseded by #48 which includes your commit as well.