From: Steve V. <vi...@ie...> - 2011-04-25 17:32:59
|
On Mon, Apr 25, 2011 at 1:29 PM, Claes Wikstrom <kl...@ta...> wrote: > On 04/25/2011 03:52 AM, Steve Vinoski wrote: >> OK, I fixed issue #50 yesterday and pushed it to github. I'm now >> looking at the existing JSON-RPC support in yaws_rpc and it too calls >> list_to_atom on received strings, which is a no-no. I'll need to fix >> that as part of the JSON-RPC 2.0 update. > > An alternative, which I'm not sure will work here, is > list_to_existing_atom/1, At Tail-f, we use that quite a lot. Hmm, great minds think alike. :-) That's exactly how I fixed it: the code (not yet pushed) first tries list_to_existing_atom and if that fails, then just returns a string. --steve |