gnodet commented on code in PR #395:
URL: https://github.com/apache/maven-filtering/pull/395#discussion_r4094931231
##########
src/main/java/org/apache/maven/shared/filtering/MavenFileFilter.java:
##########
@@ -71,4 +71,25 @@ void copyFile(
*/
void copyFile(Path from, Path to, boolean filtering, List<FilterWrapper>
filterWrappers, String encoding)
throws MavenFilteringException;
+
+ /**
+ * @param from The source file
+ * @param to The target file
+ * @param filtering true to apply filtering
+ * @param filterWrappers {@link List} of FileUtils.FilterWrapper
+ * @param encoding The encoding used during the filtering.
+ * @param gracefulBinaryHandling when {@code true}, files that cause
+ * {@link java.nio.charset.MalformedInputException} are copied
as-is with a warning
+ * instead of failing the build
+ * @throws MavenFilteringException In case of an error.
+ * @since 3.4.0
+ */
+ void copyFile(
+ Path from,
+ Path to,
+ boolean filtering,
+ List<FilterWrapper> filterWrappers,
+ String encoding,
+ boolean gracefulBinaryHandling)
+ throws MavenFilteringException;
}
Review Comment:
This is already a `default` method (line 87): it delegates to the 5-arg
overload, so no existing implementation will break. Backward compatibility is
preserved. No change needed here.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
