[sling-slingfeature-maven-plugin] branch SLING-8273 updated: SLING-8273 - APIs jar MOJO doesn't produce Javadoc if sources contain non-ascii chars

[simonetripodi]

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

simonetripodi pushed a commit to branch SLING-8273
in repository 
https://gitbox.apache.org/repos/asf/sling-slingfeature-maven-plugin.git


The following commit(s) were added to refs/heads/SLING-8273 by this push:
     new 7de76b8  SLING-8273 - APIs jar MOJO doesn't produce Javadoc if sources 
contain non-ascii chars
7de76b8 is described below

commit 7de76b86a4d37e00b80710746d7ce46d91a0bdf4
Author: Simo Tripodi <strip...@adobe.com>
AuthorDate: Wed Feb 13 17:12:16 2019 +0100

    SLING-8273 - APIs jar MOJO doesn't produce Javadoc if sources contain
    non-ascii chars
    
    properly renamed constant
---
 src/main/java/org/apache/sling/feature/maven/mojos/ApisJarMojo.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/src/main/java/org/apache/sling/feature/maven/mojos/ApisJarMojo.java 
b/src/main/java/org/apache/sling/feature/maven/mojos/ApisJarMojo.java
index 2e29a84..527f561 100644
--- a/src/main/java/org/apache/sling/feature/maven/mojos/ApisJarMojo.java
+++ b/src/main/java/org/apache/sling/feature/maven/mojos/ApisJarMojo.java
@@ -110,7 +110,7 @@ public class ApisJarMojo extends 
AbstractIncludingFeatureMojo {
 
     private static final String NON_ASCII_PATTERN = "[^\\p{ASCII}]";
 
-    private static final String EMPTY_STRING = " ";
+    private static final String SPACE = " ";
 
     @Parameter
     private FeatureSelectionConfig selection;
@@ -504,7 +504,7 @@ public class ApisJarMojo extends 
AbstractIncludingFeatureMojo {
                 // this to prevent 'unmappable character for encoding UTF8' 
error
                 if (includedFile.endsWith(JAVA_EXTENSION)) {
                     String javaSource = FileUtils.fileRead(source, 
StandardCharsets.UTF_8.name())
-                                                 
.replaceAll(NON_ASCII_PATTERN, EMPTY_STRING);
+                                                 
.replaceAll(NON_ASCII_PATTERN, SPACE);
                     FileUtils.fileWrite(target, StandardCharsets.UTF_8.name(), 
javaSource);
                 } else {
                     FileUtils.copyFile(source, target);