yup, good point


On Sat, Aug 17, 2013 at 2:02 PM, Norman Maurer <notifications@github.com>wrote:

> In transport/src/main/java/io/netty/channel/ChannelOutboundBuffer.java:
>
> > @@ -98,164 +91,65 @@ static ChannelOutboundBuffer newInstance(AbstractChannel channel) {
> >      private ChannelOutboundBuffer(Handle handle) {
> >          this.handle = handle;
> >
> > -        flushed = new Object[INITIAL_CAPACITY];
> > -        flushedPromises = new ChannelPromise[INITIAL_CAPACITY];
> > -        flushedPendingSizes = new int[INITIAL_CAPACITY];
> > -        flushedProgresses = new long[INITIAL_CAPACITY];
> > -        flushedTotals = new long[INITIAL_CAPACITY];
> > +        buffer = new Entry[INITIAL_CAPACITY];
> > +        for (int i = 0; i < INITIAL_CAPACITY; i++) {
>
> mabye use buffer.length in the loop just to be on the safe side for
> further refactoring ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/netty/netty/pull/1755/files#r5829450>
> .
>