{"action":"created","issue":{"url":"https://api.github.com/repos/emacs-helm/helm/issues/1232","labels_url":"https://api.github.com/repos/emacs-helm/helm/issues/1232/labels{/name}","comments_url":"https://api.github.com/repos/emacs-helm/helm/issues/1232/comments","events_url":"https://api.github.com/repos/emacs-helm/helm/issues/1232/events","html_url":"https://github.com/emacs-helm/helm/issues/1232","id":112256554,"number":1232,"title":"BUG: helm-match-from-candidates can exceed LIMIT when len(matchfns)>1","user":{"login":"AlKapote","id":14925408,"avatar_url":"https://avatars.githubusercontent.com/u/14925408?v=3","gravatar_id":"","url":"https://api.github.com/users/AlKapote","html_url":"https://github.com/AlKapote","followers_url":"https://api.github.com/users/AlKapote/followers","following_url":"https://api.github.com/users/AlKapote/following{/other_user}","gists_url":"https://api.github.com/users/AlKapote/gists{/gist_id}","starred_url":"https://api.github.com/users/AlKapote/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AlKapote/subscriptions","organizations_url":"https://api.github.com/users/AlKapote/orgs","repos_url":"https://api.github.com/users/AlKapote/repos","events_url":"https://api.github.com/users/AlKapote/events{/privacy}","received_events_url":"https://api.github.com/users/AlKapote/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-10-19T23:58:10Z","updated_at":"2015-10-20T20:02:43Z","closed_at":null,"body":"If the first matchfn returns LIMIT candidates, the equality check in [`helm--accumulate-candidates`](https://github.com/emacs-helm/helm/blob/master/helm.el#L3074)\r\nwill occur only after the count is incremented for the first hit on the next matchfn. At that point \r\nthe check will always fail and subsequent matchfn will return all they can.\r\nIf I change  `helm--accumulate-candidates` and move the check prior to the push the result limit is enforce, but it is also enforced in the initial candidate list passed into it (seems like `helm-search-from-candidate-buffer` is relying on this behavior?) Adding a `when` to the outer loop in [`helm-match-from-candidates`](https://github.com/emacs-helm/helm/blob/master/helm.el#L3117) looks ok.\r\n "},"comment":{"url":"https://api.github.com/repos/emacs-helm/helm/issues/comments/149684651","html_url":"https://github.com/emacs-helm/helm/issues/1232#issuecomment-149684651","issue_url":"https://api.github.com/repos/emacs-helm/helm/issues/1232","id":149684651,"user":{"login":"thierryvolpiatto","id":1533939,"avatar_url":"https://avatars.githubusercontent.com/u/1533939?v=3","gravatar_id":"","url":"https://api.github.com/users/thierryvolpiatto","html_url":"https://github.com/thierryvolpiatto","followers_url":"https://api.github.com/users/thierryvolpiatto/followers","following_url":"https://api.github.com/users/thierryvolpiatto/following{/other_user}","gists_url":"https://api.github.com/users/thierryvolpiatto/gists{/gist_id}","starred_url":"https://api.github.com/users/thierryvolpiatto/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/thierryvolpiatto/subscriptions","organizations_url":"https://api.github.com/users/thierryvolpiatto/orgs","repos_url":"https://api.github.com/users/thierryvolpiatto/repos","events_url":"https://api.github.com/users/thierryvolpiatto/events{/privacy}","received_events_url":"https://api.github.com/users/thierryvolpiatto/received_events","type":"User","site_admin":false},"created_at":"2015-10-20T20:02:43Z","updated_at":"2015-10-20T20:02:43Z","body":"\nAlKapote <notifications@github.com> writes:\n\n>             (when (< item-count limit)   ; <=======\n\nApplied, thanks.\n\n-- \nThierry\nhttps://emacs-helm.github.io/helm/\n"}}