You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by er...@apache.org on 2012/03/01 23:30:27 UTC
svn commit: r1295953 -
/commons/proper/math/trunk/src/main/java/org/apache/commons/math3/linear/SymmLQ.java
Author: erans
Date: Thu Mar 1 22:30:26 2012
New Revision: 1295953
URL: http://svn.apache.org/viewvc?rev=1295953&view=rev
Log:
Made inner class "static" because it is "Serializable" while the enclosing
class is not (bad practice signalled by "FindBugs").
Modified:
commons/proper/math/trunk/src/main/java/org/apache/commons/math3/linear/SymmLQ.java
Modified: commons/proper/math/trunk/src/main/java/org/apache/commons/math3/linear/SymmLQ.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math3/linear/SymmLQ.java?rev=1295953&r1=1295952&r2=1295953&view=diff
==============================================================================
--- commons/proper/math/trunk/src/main/java/org/apache/commons/math3/linear/SymmLQ.java (original)
+++ commons/proper/math/trunk/src/main/java/org/apache/commons/math3/linear/SymmLQ.java Thu Mar 1 22:30:26 2012
@@ -658,17 +658,17 @@ public class SymmLQ
*
* @version $Id$
*/
- private class SymmLQEvent extends IterativeLinearSolverEvent {
+ private static class SymmLQEvent extends IterativeLinearSolverEvent {
/*
* TODO This class relies dangerously on references being transparently
* updated.
*/
- /** */
+ /** Identifier. */
private static final long serialVersionUID = 2012012801L;
/** A reference to the state of this solver. */
- private final State state;
+ private final transient State state;
/**
* Creates a new instance of this class.
@@ -677,15 +677,15 @@ public class SymmLQ
* occurred
* @param state the state of this solver at the time of creation
*/
- public SymmLQEvent(final Object source, final State state) {
- super(source, getIterationManager().getIterations());
+ public SymmLQEvent(final SymmLQ source, final State state) {
+ super(source, source.getIterationManager().getIterations());
this.state = state;
}
/** {@inheritDoc} */
@Override
public int getIterations() {
- return getIterationManager().getIterations();
+ return ((SymmLQ) getSource()).getIterationManager().getIterations();
}
/** {@inheritDoc} */