On Tue, Feb 22, 2011 at 11:25:20AM -0700, Glenn Adams wrote:
> I notice also that the nightly build target does not run all the junit
> tests. It would be better if it run all of them plus checkstyle and
> findbugs.
Many junit tests require a display. Nightly builds are run in a
headless configura
On Tue, Feb 22, 2011 at 07:15:17PM +, Vincent Hennebert wrote:
> On 22/02/11 07:24, Simon Pepping wrote:
> > Not all FOP developers are willing to use findbugs. I hid the findbugs
> > errors as a courtesy to those FOP developers who do use findbugs, so
> > they can check their own code based on
https://issues.apache.org/bugzilla/show_bug.cgi?id=50483
Andreas L. Delmelle changed:
What|Removed |Added
Blocks||50723
--
Configure bugmail:
https://issues.apache.org/bugzilla/show_bug.cgi?id=50723
Andreas L. Delmelle changed:
What|Removed |Added
Depends on||50483
--- Comment #6 from An
https://issues.apache.org/bugzilla/show_bug.cgi?id=50723
--- Comment #5 from Andreas L. Delmelle 2011-02-22
18:15:02 EST ---
Looking closer, this indeed seems like a fairly inconsequential (apart from
fixing the bug, obviously ;-P).
Still, endorsing and committing it would seem to require incor
I support enabling assertions by default on junit tests. More testing is a
good thing.
G.
On Tue, Feb 22, 2011 at 3:40 PM, Andreas Delmelle <
andreas.delme...@telenet.be> wrote:
> On 22 Feb 2011, at 19:25, Andreas Delmelle wrote:
>
> > One way to enable assertions I found so far, and that seems
On 22 Feb 2011, at 19:25, Andreas Delmelle wrote:
> One way to enable assertions I found so far, and that seems to work:
> - use fork="true" on the target
> - insert: in that target
>
> That seems to yield the expected behavior at first glance. Our junit targets
> are not forked, however. Not
On 22 Feb 2011, at 22:38, adelme...@apache.org wrote:
> Author: adelmelle
> Date: Tue Feb 22 21:38:28 2011
> New Revision: 1073518
>
> @@ -402,7 +398,6 @@ public final class ColorUtil {
>
> /* Ask FOP factory to get ColorSpace for the specified ICC
> profile source */
>
https://issues.apache.org/bugzilla/show_bug.cgi?id=50723
--- Comment #4 from Peter Coppens 2011-02-22
16:44:59 EST ---
Wonder whether anyone of the fop community is willing to share viewpoints on
the approach taken in the patch. Seems a fairly safe change at first sight.
--
Configure bugmail:
On 22 Feb 2011, at 21:50, Andreas Delmelle wrote:
> ColorUtil seems to be somewhat of a can of worms...
> If I judge correctly, for some (if not most) of the methods, there seems to
> be no need for a try-catch block (?)
Sorry, forgot RuntimeExceptions, of course, which are converted into
Pro
On 22 Feb 2011, at 20:39, Andreas Delmelle wrote:
> ...
> --and so, I caved in started already...
In PDFFactory, I have some conflicts to work out first, but the fix-up is
rather simple there. In the respective methods, eliminate all unnecessary
boxing and use explicit (double) casts where appr
On 22 Feb 2011, at 20:15, Vincent Hennebert wrote:
>
> Can we revert commit 1071912 and re-consider the issues one-by-one
> before ignoring them?
+1
As far as I can see, the raised warnings are really not so challenging that
they cannot be addressed right away. Implementing an equals() method,
On 22/02/11 07:24, Simon Pepping wrote:
> Not all FOP developers are willing to use findbugs. I hid the findbugs
> errors as a courtesy to those FOP developers who do use findbugs, so
> they can check their own code based on a clean slate.
I agree that ignoring all the existing issues at the time
Since the FOP project itself does not use any IDE for builds, then the ANT
build process should be considered the standard process for compilation,
junit tests, checkstyle tests, findbugs tests, etc.
I always run:
ant clean junit checkstyle findbugs
in order to verify no reported errors before I
On 22 Feb 2011, at 19:04, Andreas Delmelle wrote:
> On 22 Feb 2011, at 08:50, Simon Pepping wrote:
>> It would be nice if assertions were checked during the junit tests.
>> However, when I set: , I get
>> junit.framework.AssertionFailedError errors. How can this be done?
>
> Sounds like a good i
On 22 Feb 2011, at 08:34, Simon Pepping wrote:
> When I build the project or part of it with Eclipse, and run findbugs
> afterwards (with ant), I get a number of errors. Now I always make a
> clean compile before running findbugs. I do not understand why Eclipse
> builds would create findbugs erro
On 22 Feb 2011, at 08:50, Simon Pepping wrote:
> On Mon, Feb 21, 2011 at 08:28:33PM +0100, Andreas Delmelle wrote:
>> I saw one exclusion --unconfirmed cast-- that would seem to stem from my
>> recent refactoring in the BlockStackingLMs. Not sure why an exclusion was
>> chosen here, but adding a
https://issues.apache.org/bugzilla/show_bug.cgi?id=49801
alpa...@gmail.com changed:
What|Removed |Added
CC||alpa...@gmail.com
--
Configure
18 matches
Mail list logo