[royale-compiler] branch develop updated: avoid generation of the .js.map file in release mode for App.js

[carlosrovira]

This is an automated email from the ASF dual-hosted git repository.

carlosrovira pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/royale-compiler.git


The following commit(s) were added to refs/heads/develop by this push:
     new ad33823  avoid generation of the .js.map file in release mode for 
App.js
ad33823 is described below

commit ad33823b2073094087ad2bd157a4c47f6e61dc2a
Author: Carlos Rovira <carlosrov...@apache.org>
AuthorDate: Sun Sep 30 20:14:48 2018 +0200

    avoid generation of the .js.map file in release mode for App.js
---
 .../internal/codegen/mxml/royale/MXMLRoyalePublisher.java    |  2 +-
 .../royale/compiler/utils/JSClosureCompilerWrapper.java      | 12 ++++++++----
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git 
a/compiler-jx/src/main/java/org/apache/royale/compiler/internal/codegen/mxml/royale/MXMLRoyalePublisher.java
 
b/compiler-jx/src/main/java/org/apache/royale/compiler/internal/codegen/mxml/royale/MXMLRoyalePublisher.java
index 6f81156..655a162 100644
--- 
a/compiler-jx/src/main/java/org/apache/royale/compiler/internal/codegen/mxml/royale/MXMLRoyalePublisher.java
+++ 
b/compiler-jx/src/main/java/org/apache/royale/compiler/internal/codegen/mxml/royale/MXMLRoyalePublisher.java
@@ -505,7 +505,7 @@ public class MXMLRoyalePublisher extends JSGoogPublisher 
implements IJSPublisher
             
compilerWrapper.setOptions(projectReleaseMainFile.getCanonicalPath(), 
useStrictPublishing, !googConfiguration.getRemoveCirculars(), projectName);
             compilerWrapper.targetFilePath = 
projectReleaseMainFile.getCanonicalPath();
 
-            compilerWrapper.compile();
+            compilerWrapper.compile(false);
 
             //we don't want sourceMap file in release build since it's only 
needed in debug
             //appendSourceMapLocation(projectReleaseMainFile, projectName);
diff --git 
a/compiler-jx/src/main/java/org/apache/royale/compiler/utils/JSClosureCompilerWrapper.java
 
b/compiler-jx/src/main/java/org/apache/royale/compiler/utils/JSClosureCompilerWrapper.java
index 5437c70..a7dd7dd 100644
--- 
a/compiler-jx/src/main/java/org/apache/royale/compiler/utils/JSClosureCompilerWrapper.java
+++ 
b/compiler-jx/src/main/java/org/apache/royale/compiler/utils/JSClosureCompilerWrapper.java
@@ -122,7 +122,8 @@ public class JSClosureCompilerWrapper
        provideds = set;
     }
     
-    public void compile()
+    // isDebug is used to avoid sourceMaps on release build
+    public void compile(Boolean isDebug)
     {
        System.out.println("list of source files");
        for (SourceFile file : jsSourceFiles_)
@@ -164,9 +165,12 @@ public class JSClosureCompilerWrapper
             targetFile.write(compiler_.toSource());
             targetFile.close();
 
-            FileWriter sourceMapFile = new 
FileWriter(options_.sourceMapOutputPath);
-            compiler_.getSourceMap().appendTo(sourceMapFile, "");
-            sourceMapFile.close();
+            if(isDebug)
+            {
+                FileWriter sourceMapFile = new 
FileWriter(options_.sourceMapOutputPath);
+                compiler_.getSourceMap().appendTo(sourceMapFile, "");
+                sourceMapFile.close();
+            }
         }
         catch (IOException error)
         {