From 6cf767528a154f27e5faf657db7774e61082423a Mon Sep 17 00:00:00 2001
From: Jeroen van Erp <jeroen@hierynomus.com>
Date: Wed, 1 Apr 2015 11:25:43 +0200
Subject: [PATCH] Added comment to check why that field is needed

---
 src/main/java/net/schmizz/sshj/sftp/RemoteFile.java | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/main/java/net/schmizz/sshj/sftp/RemoteFile.java b/src/main/java/net/schmizz/sshj/sftp/RemoteFile.java
index 318dd87f..5bb48362 100644
--- a/src/main/java/net/schmizz/sshj/sftp/RemoteFile.java
+++ b/src/main/java/net/schmizz/sshj/sftp/RemoteFile.java
@@ -88,6 +88,7 @@ public class RemoteFile
             throws IOException {
         return requester.request(newRequest(PacketType.WRITE)
                                          .putUInt64(fileOffset)
+                                        // TODO The SFTP spec claims this field is unneeded...? See #187
                                          .putUInt32(len)
                                          .putRawBytes(data, off, len)
         );