andreacavalli/netty-incubator-buffer-api
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #65 from netty/send-composite

Browse Source 
Fix composite buffer send bug
This commit is contained in:
Chris Vest 2021-05-07 12:40:20 +02:00 committed by GitHub
commit f19f04291e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/io/netty/buffer/api/CompositeBuffer.java
View File

@ -1351,7 +1351,7 @@ public final class CompositeBuffer extends RcSupport<Buffer, CompositeBuffer> im
for (int i = 0; i < sends.length; i++) {
received[i] = sends[i].receive();
}
var composite = new CompositeBuffer(allocator, received, drop, true);
var composite = new CompositeBuffer(allocator, received, drop, false);
composite.readOnly = readOnly;
drop.attach(composite);
return composite;

1
src/test/java/io/netty/buffer/api/BufferReferenceCountingTest.java
View File

@ -350,7 +350,6 @@ public class BufferReferenceCountingTest extends BufferTestSupport {
}
}
@Disabled // TODO
@ParameterizedTest
@MethodSource("allocators")
public void sliceMustBecomeOwnedOnSourceBufferClose(Fixture fixture) {