{"action":"created","issue":{"url":"https://api.github.com/repos/netty/netty/issues/8420","repository_url":"https://api.github.com/repos/netty/netty","labels_url":"https://api.github.com/repos/netty/netty/issues/8420/labels{/name}","comments_url":"https://api.github.com/repos/netty/netty/issues/8420/comments","events_url":"https://api.github.com/repos/netty/netty/issues/8420/events","html_url":"https://github.com/netty/netty/pull/8420","id":372397185,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI0NTkxMzA0","number":8420,"title":"Eliminate more array allocations related to CompositeByteBuf usage","user":{"login":"njhill","id":16958488,"node_id":"MDQ6VXNlcjE2OTU4NDg4","avatar_url":"https://avatars3.githubusercontent.com/u/16958488?v=4","gravatar_id":"","url":"https://api.github.com/users/njhill","html_url":"https://github.com/njhill","followers_url":"https://api.github.com/users/njhill/followers","following_url":"https://api.github.com/users/njhill/following{/other_user}","gists_url":"https://api.github.com/users/njhill/gists{/gist_id}","starred_url":"https://api.github.com/users/njhill/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/njhill/subscriptions","organizations_url":"https://api.github.com/users/njhill/orgs","repos_url":"https://api.github.com/users/njhill/repos","events_url":"https://api.github.com/users/njhill/events{/privacy}","received_events_url":"https://api.github.com/users/njhill/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2018-10-22T05:17:23Z","updated_at":"2018-10-22T11:02:19Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/netty/netty/pulls/8420","html_url":"https://github.com/netty/netty/pull/8420","diff_url":"https://github.com/netty/netty/pull/8420.diff","patch_url":"https://github.com/netty/netty/pull/8420.patch"},"body":"Motivation:\r\n\r\nThere are more places where allocation of arrays of buffers can be avoided.\r\n\r\nModifications:\r\n\r\n- Modify `CompositeByteBuffer.addComponents0(boolean,int,Iterable)` to use the `Iterable` directly rather than copy to an array first (and possibly to an `ArrayList` before that)\r\n- Modify `Unpooled.wrappedBuffer(byte[]...)` and `Unpooled.wrappedBuffer(ByteBuffer...)` to pass a wrapping `Iterable` rather than copy to a new `ByteBuf` array\r\n- Modify `CompositeByteBuffer.nioBuffers(int,int)` to use `RecyclableArrayList` (in line with existing `FixedCompositeByteBuf` impl)\r\n\r\nResult:\r\n\r\nLess allocation/copying when working with `CompositeByteBuf`s"},"comment":{"url":"https://api.github.com/repos/netty/netty/issues/comments/431803274","html_url":"https://github.com/netty/netty/pull/8420#issuecomment-431803274","issue_url":"https://api.github.com/repos/netty/netty/issues/8420","id":431803274,"node_id":"MDEyOklzc3VlQ29tbWVudDQzMTgwMzI3NA==","user":{"login":"netty-bot","id":39312450,"node_id":"MDQ6VXNlcjM5MzEyNDUw","avatar_url":"https://avatars0.githubusercontent.com/u/39312450?v=4","gravatar_id":"","url":"https://api.github.com/users/netty-bot","html_url":"https://github.com/netty-bot","followers_url":"https://api.github.com/users/netty-bot/followers","following_url":"https://api.github.com/users/netty-bot/following{/other_user}","gists_url":"https://api.github.com/users/netty-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/netty-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/netty-bot/subscriptions","organizations_url":"https://api.github.com/users/netty-bot/orgs","repos_url":"https://api.github.com/users/netty-bot/repos","events_url":"https://api.github.com/users/netty-bot/events{/privacy}","received_events_url":"https://api.github.com/users/netty-bot/received_events","type":"User","site_admin":false},"created_at":"2018-10-22T11:02:19Z","updated_at":"2018-10-22T11:02:19Z","author_association":"NONE","body":"Can one of the admins verify this patch?"}}
{"actor":{"display_login":"netty-bot"}}