{"action":"created","issue":{"url":"https://api.github.com/repos/DalekBaldwin/on-lisp/issues/5","labels_url":"https://api.github.com/repos/DalekBaldwin/on-lisp/issues/5/labels{/name}","comments_url":"https://api.github.com/repos/DalekBaldwin/on-lisp/issues/5/comments","events_url":"https://api.github.com/repos/DalekBaldwin/on-lisp/issues/5/events","html_url":"https://github.com/DalekBaldwin/on-lisp/pull/5","id":78223348,"number":5,"title":"Add defdelim for non-dispatching read-macros","user":{"login":"EuAndreh","id":2935736,"avatar_url":"https://avatars.githubusercontent.com/u/2935736?v=3","gravatar_id":"","url":"https://api.github.com/users/EuAndreh","html_url":"https://github.com/EuAndreh","followers_url":"https://api.github.com/users/EuAndreh/followers","following_url":"https://api.github.com/users/EuAndreh/following{/other_user}","gists_url":"https://api.github.com/users/EuAndreh/gists{/gist_id}","starred_url":"https://api.github.com/users/EuAndreh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/EuAndreh/subscriptions","organizations_url":"https://api.github.com/users/EuAndreh/orgs","repos_url":"https://api.github.com/users/EuAndreh/repos","events_url":"https://api.github.com/users/EuAndreh/events{/privacy}","received_events_url":"https://api.github.com/users/EuAndreh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-05-19T19:36:09Z","updated_at":"2015-05-21T01:03:35Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/DalekBaldwin/on-lisp/pulls/5","html_url":"https://github.com/DalekBaldwin/on-lisp/pull/5","diff_url":"https://github.com/DalekBaldwin/on-lisp/pull/5.diff","patch_url":"https://github.com/DalekBaldwin/on-lisp/pull/5.patch"},"body":"I separated the original `defdelim` into two:\r\n+ `defdelim#`: the original `defdelim`\r\n+ `defdelim`: the equivalent of the original `defdelim` but using `set-macro-character` instead of `set-dispatch-macro-character`.\r\n\r\nThe naming may confuse someone looking too fast. The names explain better the differences, but differ from the book.\r\n\r\nWhat do you think?"},"comment":{"url":"https://api.github.com/repos/DalekBaldwin/on-lisp/issues/comments/104084443","html_url":"https://github.com/DalekBaldwin/on-lisp/pull/5#issuecomment-104084443","issue_url":"https://api.github.com/repos/DalekBaldwin/on-lisp/issues/5","id":104084443,"user":{"login":"DalekBaldwin","id":1544692,"avatar_url":"https://avatars.githubusercontent.com/u/1544692?v=3","gravatar_id":"","url":"https://api.github.com/users/DalekBaldwin","html_url":"https://github.com/DalekBaldwin","followers_url":"https://api.github.com/users/DalekBaldwin/followers","following_url":"https://api.github.com/users/DalekBaldwin/following{/other_user}","gists_url":"https://api.github.com/users/DalekBaldwin/gists{/gist_id}","starred_url":"https://api.github.com/users/DalekBaldwin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DalekBaldwin/subscriptions","organizations_url":"https://api.github.com/users/DalekBaldwin/orgs","repos_url":"https://api.github.com/users/DalekBaldwin/repos","events_url":"https://api.github.com/users/DalekBaldwin/events{/privacy}","received_events_url":"https://api.github.com/users/DalekBaldwin/received_events","type":"User","site_admin":false},"created_at":"2015-05-21T01:03:35Z","updated_at":"2015-05-21T01:03:35Z","body":"I think it will only lead to feature bloat if we add a new variant of a function with no example usage, especially considering that we don't even use the original one anymore. At this point, I'd only favor new additions if they demonstrate how to adapt the code more closely to modern CL conventions, which mostly means making it easier to use them as modular components in the construction of larger systems. We're already trying to steer readers toward named-readtables as a superior solution, so it would just confuse them to add little convenience functions for such an obscure corner of the language."}}