{"action":"created","issue":{"url":"https://api.github.com/repos/gdamore/less-fork/issues/19","labels_url":"https://api.github.com/repos/gdamore/less-fork/issues/19/labels{/name}","comments_url":"https://api.github.com/repos/gdamore/less-fork/issues/19/comments","events_url":"https://api.github.com/repos/gdamore/less-fork/issues/19/events","html_url":"https://github.com/gdamore/less-fork/pull/19","id":115254832,"number":19,"title":"Forbid \"more -p h\" or \"less ++h\"","user":{"login":"nicm","id":304243,"avatar_url":"https://avatars.githubusercontent.com/u/304243?v=3","gravatar_id":"","url":"https://api.github.com/users/nicm","html_url":"https://github.com/nicm","followers_url":"https://api.github.com/users/nicm/followers","following_url":"https://api.github.com/users/nicm/following{/other_user}","gists_url":"https://api.github.com/users/nicm/gists{/gist_id}","starred_url":"https://api.github.com/users/nicm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nicm/subscriptions","organizations_url":"https://api.github.com/users/nicm/orgs","repos_url":"https://api.github.com/users/nicm/repos","events_url":"https://api.github.com/users/nicm/events{/privacy}","received_events_url":"https://api.github.com/users/nicm/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-11-05T10:32:44Z","updated_at":"2015-11-05T18:43:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/gdamore/less-fork/pulls/19","html_url":"https://github.com/gdamore/less-fork/pull/19","diff_url":"https://github.com/gdamore/less-fork/pull/19.diff","patch_url":"https://github.com/gdamore/less-fork/pull/19.patch"},"body":"Do not allow h as part of -p (for more) or ++ (for less) because it will just end up in a loop (it is every_first_cmd, which is unget'ed for every new file). For example: less ++hq /etc/hosts\r\n\r\nFrom OpenBSD, original commit message was:\r\n````\r\n----------------------------\r\nrevision 1.12\r\ndate: 2014/04/14 20:10:33;  author: schwarze;  state: Exp;  lines: +7 -0;\r\nDeny requests to display interactive help unless we can be really sure\r\nthat they actually result from the user interactively asking for help.\r\nHelp requests originating from various other sources caused infinite loops.\r\nOK millert@ \"better than getting stuck in a help loop (sweet irony ;)\" jmc@\r\n----------------------------"},"comment":{"url":"https://api.github.com/repos/gdamore/less-fork/issues/comments/154149256","html_url":"https://github.com/gdamore/less-fork/pull/19#issuecomment-154149256","issue_url":"https://api.github.com/repos/gdamore/less-fork/issues/19","id":154149256,"user":{"login":"gdamore","id":1295229,"avatar_url":"https://avatars.githubusercontent.com/u/1295229?v=3","gravatar_id":"","url":"https://api.github.com/users/gdamore","html_url":"https://github.com/gdamore","followers_url":"https://api.github.com/users/gdamore/followers","following_url":"https://api.github.com/users/gdamore/following{/other_user}","gists_url":"https://api.github.com/users/gdamore/gists{/gist_id}","starred_url":"https://api.github.com/users/gdamore/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gdamore/subscriptions","organizations_url":"https://api.github.com/users/gdamore/orgs","repos_url":"https://api.github.com/users/gdamore/repos","events_url":"https://api.github.com/users/gdamore/events{/privacy}","received_events_url":"https://api.github.com/users/gdamore/received_events","type":"User","site_admin":false},"created_at":"2015-11-05T18:43:53Z","updated_at":"2015-11-05T18:43:53Z","body":"So I was unable to reproduce the hang with LESS_IS_MORE  and -p h.   I did reproduce it with ++h.\r\n\r\nAre we sure that we need this in all cases?  I'm going to go ahead and merge this, because frankly anyone using more -p h deserves heartache.  But I think we might actually be violating the POSIX standard here.  "}}