{"action":"created","issue":{"url":"https://api.github.com/repos/asciidoctor/asciidoctor/issues/1638","labels_url":"https://api.github.com/repos/asciidoctor/asciidoctor/issues/1638/labels{/name}","comments_url":"https://api.github.com/repos/asciidoctor/asciidoctor/issues/1638/comments","events_url":"https://api.github.com/repos/asciidoctor/asciidoctor/issues/1638/events","html_url":"https://github.com/asciidoctor/asciidoctor/pull/1638","id":124077715,"number":1638,"title":"Remove horizontal space after .URL's third argument…","user":{"login":"ds26gte","id":3708729,"avatar_url":"https://avatars.githubusercontent.com/u/3708729?v=3","gravatar_id":"","url":"https://api.github.com/users/ds26gte","html_url":"https://github.com/ds26gte","followers_url":"https://api.github.com/users/ds26gte/followers","following_url":"https://api.github.com/users/ds26gte/following{/other_user}","gists_url":"https://api.github.com/users/ds26gte/gists{/gist_id}","starred_url":"https://api.github.com/users/ds26gte/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ds26gte/subscriptions","organizations_url":"https://api.github.com/users/ds26gte/orgs","repos_url":"https://api.github.com/users/ds26gte/repos","events_url":"https://api.github.com/users/ds26gte/events{/privacy}","received_events_url":"https://api.github.com/users/ds26gte/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-12-28T15:00:03Z","updated_at":"2015-12-29T18:03:06Z","closed_at":"2015-12-29T17:36:23Z","pull_request":{"url":"https://api.github.com/repos/asciidoctor/asciidoctor/pulls/1638","html_url":"https://github.com/asciidoctor/asciidoctor/pull/1638","diff_url":"https://github.com/asciidoctor/asciidoctor/pull/1638.diff","patch_url":"https://github.com/asciidoctor/asciidoctor/pull/1638.patch"},"body":"…since linefeed after \\c is already a space.\r\n\r\nThis is a subtle bug that occurs at only some line lengths. E.g.,\r\n```\r\nThe script troff2page runs on the Unix command-line on Cygwin, Linux,\r\nMac OS X, and Solaris.  It uses the Common Lisp implementation mentioned\r\nin the shell environment variable LISP, which can currently be set to\r\neither clisp, clozure, cmu, ecl or sbcl: the corresponding Lisp\r\nimplementations being\r\nhttp://clisp.sf.net[CLISP],\r\nhttp://ccl.clozure.com[Clozure CL],\r\nhttp://cmucl.org[CMUCL],\r\nhttp://ecls.sf.net[ECL], and\r\nhttp://sbcl.sf.net[SBCL].\r\n```\r\nThe line `http://ecls.sf.net[ECL], and` currently gets translated to\r\n```\r\n.URL \"http://ecls.sf.net\" \"ECL\" \",\"\r\n\\c\r\n and\r\n```\r\nNote the space before `and`. If the man pager happens to line break just at the `,`, that space shows up as a glaring leading space in the immediately following line.  (My example above will show up this error if you have a `MANWIDTH` of 80.)\r\n\r\nThe translation is placing after the generated `\\c` whatever's after `.URL`'s 3rd argument, and this typically has some leading space.  But this isn't needed (indeed, _shouldn't_ be included), because the linefeed introduced after the generated `\\c` is already a space. (The extraneous space shows up even when there is no coincidental linebreak, but is less visible to human eyes then because it seems like it could be an artifact of the rather courageous monospace-based both-sides line justification done by the man pager.)\r\n\r\nPR submitted."},"comment":{"url":"https://api.github.com/repos/asciidoctor/asciidoctor/issues/comments/167844846","html_url":"https://github.com/asciidoctor/asciidoctor/pull/1638#issuecomment-167844846","issue_url":"https://api.github.com/repos/asciidoctor/asciidoctor/issues/1638","id":167844846,"user":{"login":"ds26gte","id":3708729,"avatar_url":"https://avatars.githubusercontent.com/u/3708729?v=3","gravatar_id":"","url":"https://api.github.com/users/ds26gte","html_url":"https://github.com/ds26gte","followers_url":"https://api.github.com/users/ds26gte/followers","following_url":"https://api.github.com/users/ds26gte/following{/other_user}","gists_url":"https://api.github.com/users/ds26gte/gists{/gist_id}","starred_url":"https://api.github.com/users/ds26gte/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ds26gte/subscriptions","organizations_url":"https://api.github.com/users/ds26gte/orgs","repos_url":"https://api.github.com/users/ds26gte/repos","events_url":"https://api.github.com/users/ds26gte/events{/privacy}","received_events_url":"https://api.github.com/users/ds26gte/received_events","type":"User","site_admin":false},"created_at":"2015-12-29T18:03:06Z","updated_at":"2015-12-29T18:03:06Z","body":"Unfortunately, in a snow-induced brain-freeze (and also pushing the limits of my git skills), I created a new `ds26gte:manpage-url-fix` rebasing it off your new `master` (instead of just rebasing `ds26gte:manpage-default-boilerplate` and leaving this one alone).\r\n\r\nI've resubmitted the PR, but there's a slight possibility that something got lost. Could you re-send your PR-to-my-PR without modifying anything so I can make sure I didn't lose anything?"}}