You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ta...@apache.org on 2019/12/09 13:26:31 UTC

[myfaces] branch master updated: MYFACES-4313

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

tandraschko pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/myfaces.git


The following commit(s) were added to refs/heads/master by this push:
     new 573912c  MYFACES-4313
573912c is described below

commit 573912c78935fb0c790136ff8715cd0bb9aa40e1
Author: Thomas Andraschko <ta...@apache.org>
AuthorDate: Mon Dec 9 14:26:24 2019 +0100

    MYFACES-4313
---
 .../org/apache/myfaces/config/DefaultFacesConfigurationProvider.java   | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/impl/src/main/java/org/apache/myfaces/config/DefaultFacesConfigurationProvider.java b/impl/src/main/java/org/apache/myfaces/config/DefaultFacesConfigurationProvider.java
index ab08bc3..9a3ca38 100644
--- a/impl/src/main/java/org/apache/myfaces/config/DefaultFacesConfigurationProvider.java
+++ b/impl/src/main/java/org/apache/myfaces/config/DefaultFacesConfigurationProvider.java
@@ -377,8 +377,7 @@ public class DefaultFacesConfigurationProvider extends FacesConfigurationProvide
     {
         try
         {
-            // just return a empty FacesConfigImpl if the file is empty
-            // we just do a shortcut here, otherwise validateXML would log a error/exception
+            // skip parsing/validating a empty faces-config.xml to avoid warn/error logs
             try (InputStream stream = ectx.getResourceAsStream(DEFAULT_FACES_CONFIG))
             {
                 if (stream != null)