You are viewing a plain text version of this content. The canonical link for it is here.
Posted to doxia-commits@maven.apache.org by lt...@apache.org on 2009/04/09 20:13:14 UTC
svn commit: r763758 - in /maven/doxia/doxia-sitetools/trunk:
doxia-doc-renderer/src/main/java/org/apache/maven/doxia/docrenderer/itext/
doxia-doc-renderer/src/main/java/org/apache/maven/doxia/docrenderer/pdf/itext/
doxia-site-renderer/src/main/java/org...
Author: ltheussl
Date: Thu Apr 9 18:13:14 2009
New Revision: 763758
URL: http://svn.apache.org/viewvc?rev=763758&view=rev
Log:
Checkstyle fixes
Modified:
maven/doxia/doxia-sitetools/trunk/doxia-doc-renderer/src/main/java/org/apache/maven/doxia/docrenderer/itext/AbstractITextRender.java
maven/doxia/doxia-sitetools/trunk/doxia-doc-renderer/src/main/java/org/apache/maven/doxia/docrenderer/pdf/itext/ITextPdfRenderer.java
maven/doxia/doxia-sitetools/trunk/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DefaultSiteRenderer.java
maven/doxia/doxia-sitetools/trunk/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DocumentRenderer.java
Modified: maven/doxia/doxia-sitetools/trunk/doxia-doc-renderer/src/main/java/org/apache/maven/doxia/docrenderer/itext/AbstractITextRender.java
URL: http://svn.apache.org/viewvc/maven/doxia/doxia-sitetools/trunk/doxia-doc-renderer/src/main/java/org/apache/maven/doxia/docrenderer/itext/AbstractITextRender.java?rev=763758&r1=763757&r2=763758&view=diff
==============================================================================
--- maven/doxia/doxia-sitetools/trunk/doxia-doc-renderer/src/main/java/org/apache/maven/doxia/docrenderer/itext/AbstractITextRender.java (original)
+++ maven/doxia/doxia-sitetools/trunk/doxia-doc-renderer/src/main/java/org/apache/maven/doxia/docrenderer/itext/AbstractITextRender.java Thu Apr 9 18:13:14 2009
@@ -80,7 +80,7 @@
extends AbstractLogEnabled
implements DocRenderer
{
- private static String XSLT_RESOURCE = "org/apache/maven/doxia/docrenderer/itext/xslt/TOC.xslt";
+ private static final String XSLT_RESOURCE = "org/apache/maven/doxia/docrenderer/itext/xslt/TOC.xslt";
private static final TransformerFactory TRANSFORMER_FACTORY = TransformerFactory.newInstance();
@@ -300,7 +300,7 @@
Writer writer = WriterFactory.newXmlWriter( outputITextFile );
ITextSink sink = (ITextSink) new ITextSinkFactory().createSink( writer );
- sink.setClassLoader( new URLClassLoader( new URL[] { outputITextFile.getParentFile().toURL() } ) );
+ sink.setClassLoader( new URLClassLoader( new URL[] { outputITextFile.getParentFile().toURI().toURL() } ) );
Reader reader = null;
try
@@ -320,11 +320,13 @@
}
catch ( ParserNotFoundException e )
{
- throw new DocumentRendererException( "Error getting a parser for '" + fullPathDoc + "': " + e.getMessage() );
+ throw new DocumentRendererException( "Error getting a parser for '"
+ + fullPathDoc + "': " + e.getMessage() );
}
catch ( ParseException e )
{
- throw new DocumentRendererException( "Error parsing '" + fullPathDoc + "': line [" + e.getLineNumber() + "] " + e.getMessage(), e );
+ throw new DocumentRendererException( "Error parsing '"
+ + fullPathDoc + "': line [" + e.getLineNumber() + "] " + e.getMessage(), e );
}
finally
{
@@ -384,7 +386,8 @@
}
catch ( DOMException e )
{
- throw new DocumentRendererException( "Error appending chapter for " + iTextFile + " : " + e.getMessage() );
+ throw new DocumentRendererException( "Error appending chapter for "
+ + iTextFile + " : " + e.getMessage() );
}
}
@@ -489,7 +492,8 @@
}
catch ( TransformerException e )
{
- throw new DocumentRendererException( "Error transformer Document from " + document + ": " + e.getMessage() );
+ throw new DocumentRendererException( "Error transformer Document from "
+ + document + ": " + e.getMessage() );
}
}
}
Modified: maven/doxia/doxia-sitetools/trunk/doxia-doc-renderer/src/main/java/org/apache/maven/doxia/docrenderer/pdf/itext/ITextPdfRenderer.java
URL: http://svn.apache.org/viewvc/maven/doxia/doxia-sitetools/trunk/doxia-doc-renderer/src/main/java/org/apache/maven/doxia/docrenderer/pdf/itext/ITextPdfRenderer.java?rev=763758&r1=763757&r2=763758&view=diff
==============================================================================
--- maven/doxia/doxia-sitetools/trunk/doxia-doc-renderer/src/main/java/org/apache/maven/doxia/docrenderer/pdf/itext/ITextPdfRenderer.java (original)
+++ maven/doxia/doxia-sitetools/trunk/doxia-doc-renderer/src/main/java/org/apache/maven/doxia/docrenderer/pdf/itext/ITextPdfRenderer.java Thu Apr 9 18:13:14 2009
@@ -84,7 +84,7 @@
}
/** {@inheritDoc} */
- public void generatePdf( File iTextFile, File pdfFile )
+ public void generatePdf( File inputFile, File pdfFile )
throws DocumentRendererException
{
if ( getLogger().isDebugEnabled() )
@@ -94,15 +94,15 @@
try
{
- ITextUtil.writePdf( new FileInputStream( iTextFile ), new FileOutputStream( pdfFile ) );
+ ITextUtil.writePdf( new FileInputStream( inputFile ), new FileOutputStream( pdfFile ) );
}
catch ( IOException e )
{
- throw new DocumentRendererException( "Cannot create PDF from " + iTextFile + ": " + e.getMessage(), e );
+ throw new DocumentRendererException( "Cannot create PDF from " + inputFile + ": " + e.getMessage(), e );
}
catch ( RuntimeException e )
{
- throw new DocumentRendererException( "Error creating PDF from " + iTextFile + ": " + e.getMessage(), e );
+ throw new DocumentRendererException( "Error creating PDF from " + inputFile + ": " + e.getMessage(), e );
}
}
@@ -251,7 +251,8 @@
}
catch ( DOMException e )
{
- throw new DocumentRendererException( "Error appending chapter for " + iTextFile + " : " + e.getMessage() );
+ throw new DocumentRendererException( "Error appending chapter for "
+ + iTextFile + " : " + e.getMessage() );
}
}
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=763758&r1=763757&r2=763758&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 Thu Apr 9 18:13:14 2009
@@ -212,13 +212,15 @@
docs.addAll( FileUtils.getFileNames( moduleBasedir, "**/*." + module.getExtension(), excludes, false ) );
// download.apt.vm
- docs.addAll( FileUtils.getFileNames( moduleBasedir, "**/*." + module.getExtension() + ".vm", excludes, false ) );
+ docs.addAll( FileUtils.getFileNames( moduleBasedir,
+ "**/*." + module.getExtension() + ".vm", excludes, false ) );
for ( Iterator k = docs.iterator(); k.hasNext(); )
{
String doc = (String) k.next();
- RenderingContext context = new RenderingContext( moduleBasedir, doc, module.getParserId(), module.getExtension() );
+ RenderingContext context =
+ new RenderingContext( moduleBasedir, doc, module.getParserId(), module.getExtension() );
// TODO: DOXIA-111: we need a general filter here that knows how to alter the context
if ( doc.endsWith( ".vm" ) )
@@ -395,7 +397,8 @@
}
catch ( ParseException e )
{
- throw new RendererException( "Error parsing '" + doc + "': line [" + e.getLineNumber() + "] " + e.getMessage(), e );
+ throw new RendererException( "Error parsing '"
+ + doc + "': line [" + e.getLineNumber() + "] " + e.getMessage(), e );
}
catch ( IOException e )
{
Modified: maven/doxia/doxia-sitetools/trunk/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DocumentRenderer.java
URL: http://svn.apache.org/viewvc/maven/doxia/doxia-sitetools/trunk/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DocumentRenderer.java?rev=763758&r1=763757&r2=763758&view=diff
==============================================================================
--- maven/doxia/doxia-sitetools/trunk/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DocumentRenderer.java (original)
+++ maven/doxia/doxia-sitetools/trunk/doxia-site-renderer/src/main/java/org/apache/maven/doxia/siterenderer/DocumentRenderer.java Thu Apr 9 18:13:14 2009
@@ -36,9 +36,9 @@
/**
* Render a document.
*
- * @param writer
- * @param renderer
- * @param siteRenderingContext
+ * @param writer the Writer.
+ * @param renderer the Renderer.
+ * @param siteRenderingContext the SiteRenderingContext.
* @throws org.apache.maven.doxia.siterenderer.RendererException if it bombs.
* @throws java.io.FileNotFoundException if it bombs.
* @throws java.io.UnsupportedEncodingException if it bombs.