[jscover] 25/69: Avoid redundant looping

[Sylvestre Ledru]

This is an automated email from the git hooks/post-receive script.

sylvestre pushed a commit to branch master
in repository jscover.

commit af519aa6200d9a817f5abf44d86f7222cad89338
Author: tntim96 <tntim96 at gmail.com>
Date:   Mon Jun 23 18:13:14 2014 +1000

    Avoid redundant looping
---
 src/main/java/jscover/server/InstrumentingRequestHandler.java | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/main/java/jscover/server/InstrumentingRequestHandler.java b/src/main/java/jscover/server/InstrumentingRequestHandler.java
index 5bb1e61..f2856ca 100644
--- a/src/main/java/jscover/server/InstrumentingRequestHandler.java
+++ b/src/main/java/jscover/server/InstrumentingRequestHandler.java
@@ -408,11 +408,11 @@ public class InstrumentingRequestHandler extends HttpServer {
             if (configuration.isIncludeUnloadedJS()) {
                 logger.fine("Searching for unloaded JavaScript");
                 unloadJSData = unloadedSourceProcessor.getEmptyCoverageData(uris.keySet());
-                for (ScriptCoverageCount scriptLinesAndSource : unloadJSData) {
-                    File src = new File(configuration.getDocumentRoot(), scriptLinesAndSource.getUri());
-                    if (!configuration.isSaveJSONOnly())
+                if (!configuration.isSaveJSONOnly())
+                    for (ScriptCoverageCount scriptLinesAndSource : unloadJSData) {
+                        File src = new File(configuration.getDocumentRoot(), scriptLinesAndSource.getUri());
                         ioUtils.copy(src, new File(reportDir, Main.reportSrcSubDir + scriptLinesAndSource.getUri()));
-                }
+                    }
             }
             long skipped = request.getInputStream().skip(request.getPostIndex());
             assert skipped == request.getPostIndex();

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-java/jscover.git