Re: [PATCH] ice-9: Add JSON module.

2015-09-22 Thread Matt Wette
> On Sep 21, 2015, at 11:49 PM, Mark H Weaver wrote: > >> 2. The comments say integers are converted to exact and floating point >> to inexact, but the code will convert 1.0 to exact. > > In Scheme terminology, 1.0 is an integer. But in guile-2.0.11: scheme@(guile-user)> (exact? (string->numb

Re: [PATCH] ice-9: Add JSON module.

2015-09-21 Thread Mark H Weaver
Matt Wette writes: > On Sep 21, 2015, at 1:08 PM, Ludovic Courtès wrote: > > > David Thompson skribis: > > JSON is an commonly encountered format when writing web > applications, > much like XML, and I think it would be a good idea if the core >

Re: [PATCH] ice-9: Add JSON module.

2015-09-21 Thread Mark H Weaver
uot;hacking", > "cycling", > "surfing" > ] > } > > Thanks to Mark Weaver and Chris Webber for helping come to a consensus > on a good syntax for JSON objects. > > From 2d4d8607aedaede98f413a84f135d8798d506233 Mon Sep 17 00

Re: [PATCH] ice-9: Add JSON module.

2015-09-21 Thread Matt Wette
> On Sep 21, 2015, at 1:08 PM, Ludovic Courtès wrote: > > David Thompson skribis: > >> JSON is an commonly encountered format when writing web applications, >> much like XML, and I think it would be a good idea if the core Guile >> distribution had an SXML equivalent for JSON. This patch intr

Re: [PATCH] ice-9: Add JSON module.

2015-09-21 Thread Ludovic Courtès
David Thompson skribis: > JSON is an commonly encountered format when writing web applications, > much like XML, and I think it would be a good idea if the core Guile > distribution had an SXML equivalent for JSON. This patch introduces > such an interface in the (ice-9 json) module. There’s al

Re: [PATCH] ice-9: Add JSON module.

2015-08-17 Thread Christopher Allan Webber
I tested this, including with the fixes specified in the email. (ice-9 json) seems to be working great... I'd love to see it merged!

Re: [PATCH] ice-9: Add JSON module.

2015-08-15 Thread David Thompson
Noticed a couple of small issues after I sent the initial patch that I've fixed in my local git branch: David Thompson writes: > +(define-module (ice-9 json) > + #:use-module (ice-9 match) > + #:use-module (srfi srfi-1) > + #:export (read-json write-json)) No need to import SRFI-1. > +(defi

[PATCH] ice-9: Add JSON module.

2015-08-15 Thread David Thompson
413a84f135d8798d506233 Mon Sep 17 00:00:00 2001 From: David Thompson Date: Sat, 15 Aug 2015 14:09:23 -0400 Subject: [PATCH] ice-9: Add JSON module. * module/ice-9/json.scm: New file. * module/Makefile.am (ICE_9_SOURCES): Add it. * test-suite/tests/json.test: New file. * test-suite/Makefile.a