Fix the issue where scp transfer completion tracking wasn't being triggered for 100% completion

This closes #91 pull request at github/apache/ant

CONTRIBUTORS

@@ -441,6 +441,7 @@ Vladislav Bauer
 Volker Leidl
 Waldek Herka
 Wang Weijun
+Wenjing Wang
 Will Wang
 William Bernardet
 William Ferguson

WHATSNEW

@@ -56,6 +56,10 @@ Fixed bugs:
    actual address could potentially be unreachable. This is now fixed
    and the resolved address is actually checked for reachability.
 
+ * Fixes an issue where scp transfer completion tracking wasn't being
+   triggered for 100% completion.
+   Github Pull Request #91
+
 
 Other changes:
 --------------

contributors.xml

@@ -1815,6 +1815,10 @@
     <first>Wang</first>
     <last>Weijun</last>
   </name>
+  <name>
+    <first>Wenjing</first>
+    <last>Wang</last>
+  </name>
   <name>
     <first>Will</first>
     <last>Wang</last>

src/main/org/apache/tools/ant/taskdefs/optional/ssh/ScpFromMessage.java

@@ -286,15 +286,14 @@ public class ScpFromMessage extends AbstractSshMessage {
                 fos.write(buf, 0, length);
                 filesize -= length;
                 totalLength += length;
-                if (filesize == 0) {
-                    break;
-                }
-
                 if (trackProgress) {
                     percentTransmitted = trackProgress(initFilesize,
                                                        totalLength,
                                                        percentTransmitted);
                 }
+                if (filesize == 0) {
+                    break;
+                }
             }
         } finally {
             final long endTime = System.currentTimeMillis();