You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@maven.apache.org by lt...@apache.org on 2013/01/22 08:09:16 UTC

svn commit: r1436769 - /maven/doxia/doxia-sitetools/trunk/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DefaultSiteRenderer.java

Author: ltheussl
Date: Tue Jan 22 07:09:15 2013
New Revision: 1436769

URL: http://svn.apache.org/viewvc?rev=1436769&view=rev
Log:
[DOXIASITETOOLS-79] better error message in case velocity parsing fails, avoid NPE

Modified:
    maven/doxia/doxia-sitetools/trunk/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DefaultSiteRenderer.java

Modified: maven/doxia/doxia-sitetools/trunk/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DefaultSiteRenderer.java
URL: http://svn.apache.org/viewvc/maven/doxia/doxia-sitetools/trunk/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DefaultSiteRenderer.java?rev=1436769&r1=1436768&r2=1436769&view=diff
==============================================================================
--- maven/doxia/doxia-sitetools/trunk/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DefaultSiteRenderer.java (original)
+++ maven/doxia/doxia-sitetools/trunk/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DefaultSiteRenderer.java Tue Jan 22 07:09:15 2013
@@ -398,6 +398,11 @@ public class DefaultSiteRenderer
                 }
             }
             sink.enableLogging( new PlexusLoggerWrapper( getLogger() ) );
+
+            if ( reader == null ) // can happen if velocity throws above
+            {
+                throw new RendererException( "Error getting a parser for '" + doc );
+            }
             doxia.parse( reader, renderingContext.getParserId(), sink );
         }
         catch ( ParserNotFoundException e )