{"action":"created","issue":{"url":"https://api.github.com/repos/hanwen/go-fuse/issues/133","repository_url":"https://api.github.com/repos/hanwen/go-fuse","labels_url":"https://api.github.com/repos/hanwen/go-fuse/issues/133/labels{/name}","comments_url":"https://api.github.com/repos/hanwen/go-fuse/issues/133/comments","events_url":"https://api.github.com/repos/hanwen/go-fuse/issues/133/events","html_url":"https://github.com/hanwen/go-fuse/pull/133","id":180294573,"number":133,"title":"Loopback: Making the code for Read() consistent with the code for Write()","user":{"login":"chmduquesne","id":234544,"avatar_url":"https://avatars.githubusercontent.com/u/234544?v=3","gravatar_id":"","url":"https://api.github.com/users/chmduquesne","html_url":"https://github.com/chmduquesne","followers_url":"https://api.github.com/users/chmduquesne/followers","following_url":"https://api.github.com/users/chmduquesne/following{/other_user}","gists_url":"https://api.github.com/users/chmduquesne/gists{/gist_id}","starred_url":"https://api.github.com/users/chmduquesne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chmduquesne/subscriptions","organizations_url":"https://api.github.com/users/chmduquesne/orgs","repos_url":"https://api.github.com/users/chmduquesne/repos","events_url":"https://api.github.com/users/chmduquesne/events{/privacy}","received_events_url":"https://api.github.com/users/chmduquesne/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-09-30T12:41:55Z","updated_at":"2016-10-04T20:41:14Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hanwen/go-fuse/pulls/133","html_url":"https://github.com/hanwen/go-fuse/pull/133","diff_url":"https://github.com/hanwen/go-fuse/pull/133.diff","patch_url":"https://github.com/hanwen/go-fuse/pull/133.patch"},"body":"In issue #132, I was wondering why the read buffer was not filled in the Read() function. Now that I understand better what is going on, I would like to suggest this patch, which IMHO makes the code more understandable.\r\n\r\nI think this version is simpler for the newcomer, because one can see how the result is filled directly from the Read() function, rather than having to dive in the code of readResultFd.\r\n\r\nFeel free to take the patch, unless you believe this does not improve the readability."},"comment":{"url":"https://api.github.com/repos/hanwen/go-fuse/issues/comments/251506790","html_url":"https://github.com/hanwen/go-fuse/pull/133#issuecomment-251506790","issue_url":"https://api.github.com/repos/hanwen/go-fuse/issues/133","id":251506790,"user":{"login":"chmduquesne","id":234544,"avatar_url":"https://avatars.githubusercontent.com/u/234544?v=3","gravatar_id":"","url":"https://api.github.com/users/chmduquesne","html_url":"https://github.com/chmduquesne","followers_url":"https://api.github.com/users/chmduquesne/followers","following_url":"https://api.github.com/users/chmduquesne/following{/other_user}","gists_url":"https://api.github.com/users/chmduquesne/gists{/gist_id}","starred_url":"https://api.github.com/users/chmduquesne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chmduquesne/subscriptions","organizations_url":"https://api.github.com/users/chmduquesne/orgs","repos_url":"https://api.github.com/users/chmduquesne/repos","events_url":"https://api.github.com/users/chmduquesne/events{/privacy}","received_events_url":"https://api.github.com/users/chmduquesne/received_events","type":"User","site_admin":false},"created_at":"2016-10-04T20:41:14Z","updated_at":"2016-10-04T20:41:14Z","body":"Hi,\r\n\r\nI am also contributing a .travis.yml for continuous integration. Because who does not like continuous integration?\r\n\r\nAnd travis [supports it](https://github.com/travis-ci/travis-ci/issues/1100#issuecomment-160169121) now (tested on another project of mine which uses go-fuse)."}}
{"actor":{"display_login":"chmduquesne"}}