Re: [xwiki-notifications] r24508 - in platform/core/trunk/xwiki-officeimporter: . src/main/java/org/xwiki/officeimporter/internal src/main/java/org/xwiki/officeimporter/internal/builder src/main/java/org/xwiki/officeimporter/internal/cleaner src/main/java/org/xwiki/officeimporter/internal/filter src/main/java/org/xwiki/officeimporter/internal/openoffice src/main/java/org/xwiki/officeimporter/internal/splitter src/main/java/org/xwiki/officeimporter/openoffice src/main/java/org/xwiki/officeimporter/splitter

1 message Options
Embed this post
Permalink
Sergiu Dumitriu-2

Re: [xwiki-notifications] r24508 - in platform/core/trunk/xwiki-officeimporter: . src/main/java/org/xwiki/officeimporter/internal src/main/java/org/xwiki/officeimporter/internal/builder src/main/java/org/xwiki/officeimporter/internal/cleaner src/main/java/org/xwiki/officeimporter/internal/filter src/main/java/org/xwiki/officeimporter/internal/openoffice src/main/java/org/xwiki/officeimporter/internal/splitter src/main/java/org/xwiki/officeimporter/openoffice src/main/java/org/xwiki/officeimporter/splitter

Reply Threaded More More options
Print post
Permalink
On 10/15/2009 12:05 PM, asiri (SVN) wrote:

> Author: asiri
> Date: 2009-10-15 12:05:22 +0200 (Thu, 15 Oct 2009)
> New Revision: 24508
>
> Modified:
> Log:
> [misc] Activated checkstyle for officeimporter code base.
>
> - Fixed 71/72 Checkstyles reported.
>
> - The remaining checkstyle issue (class fan-out ~ 22, max allowed is 20) is bit hard to fix without refactoring the officeimporter API a little.

You can fix it by removing the readStream utility method, which can be
replaced with something from commons-ioutils. This will remove all
usages of ByteArrayOutputStream and IOException and add one class from
ioutils.

--
Sergiu Dumitriu
http://purl.org/net/sergiu/
_______________________________________________
devs mailing list
[hidden email]
http://lists.xwiki.org/mailman/listinfo/devs