On Fri, Jul 5, 2013 at 6:50 PM, <e...@racket-lang.org> wrote: > > > f90fe4c Eli Barzilay <e...@racket-lang.org> 2013-07-05 18:08 > : > | Get rid of the `#:function' keyword. > | > | These problems are always dealt with via an internal function instead of > | making the name argument part of the visible API. > : > M pkgs/racket-pkgs/racket-doc/json/json.scrbl | 10 ++------ > M racket/lib/collects/json/main.rkt | 33 > +++++++++++++++-----------
I think we should keep the name argument part of the API. Imagine that I had a another function that used `read-json` or `write-json` internally, and I wanted the same good error messages that `jsexpr->bytes` has. Then I'd have to either perform string replacement on the exception, or create do something else equally ugly. In general, this is why we have things like `class/derived` -- so that wrappers can have good error messages with their names. Sam _________________________ Racket Developers list: http://lists.racket-lang.org/dev