/*
 * Copyright (c) 2012-2014 Spotify AB
 *
 * Licensed under the Apache License, Version 2.0 (the "License"); you may not
 * use this file except in compliance with the License. You may obtain a copy of
 * the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
 * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
 * License for the specific language governing permissions and limitations under
 * the License.
 */

package com.spotify.netty4.handler.codec.zmtp;

import java.util.List;

import io.netty.buffer.ByteBuf;

public class DefaultZMTPMessageEncoder implements ZMTPMessageEncoder {

  private boolean enveloped;

  public DefaultZMTPMessageEncoder(final boolean enveloped) {
    this.enveloped = enveloped;
  }

  @Override
  public void estimate(final Object msg, final ZMTPEstimator estimator) {
    final ZMTPMessage message = (ZMTPMessage) msg;

    final List<ZMTPFrame> envelope = message.envelope();
    for (int i = 0; i < envelope.size(); i++) {
      final ZMTPFrame frame = envelope.get(i);
      estimator.frame(frame.size());
    }

    if (enveloped) {
      estimator.frame(0);
    }

    final List<ZMTPFrame> content = message.content();
    for (int i = 0; i < content.size(); i++) {
      final ZMTPFrame frame = content.get(i);
      estimator.frame(frame.size());
    }
  }

  @Override
  public void encode(final Object msg, final ZMTPWriter writer) {
    final ZMTPMessage message = (ZMTPMessage) msg;

    for (int i = 0; i < message.envelope().size(); i++) {
      final ZMTPFrame frame = message.envelope().get(i);
      final ByteBuf dst = writer.frame(frame.size(), true);
      final ByteBuf src = frame.content();
      dst.writeBytes(src, src.readerIndex(), src.readableBytes());
    }

    if (enveloped) {
      writer.frame(0, true);
    }

    for (int i = 0; i < message.content().size(); i++) {
      final ZMTPFrame frame = message.content().get(i);
      final boolean more = i < message.content().size() - 1;
      final ByteBuf dst = writer.frame(frame.size(), more);
      final ByteBuf src = frame.content();
      dst.writeBytes(src, src.readerIndex(), src.readableBytes());
    }
  }
}

@@ -1107,53 +1107,8 @@
 or (
-int i = 0; i %3C envelope.size(); i++) %7B%0A      
 fina
@@ -1125,17 +1125,17 @@
 e frame 
-=
+:
  envelop
@@ -1135,24 +1135,19 @@
 envelope
-.get(i);
+) %7B
 %0A      e
@@ -1305,52 +1305,8 @@
 or (
-int i = 0; i %3C content.size(); i++) %7B%0A      
 fina
@@ -1327,17 +1327,17 @@
 ame 
-=
+:
  content
 .get
@@ -1332,24 +1332,19 @@
  content
-.get(i);
+) %7B
 %0A      e

use foreach