On Sun, 27 Oct 2013 09:52:44 -0000, l...@apache.org wrote:
Author: luc
Date: Sun Oct 27 09:52:44 2013
New Revision: 1536073

URL: http://svn.apache.org/r1536073
Log:
Added SparseGradient to deal efficiently with numerous variables.

[...]
+
+    /**
+     * Multiply in place.
+     * <p>
+     * This method is designed to be faster when used multiple times
in a loop.
+     * </p>
+     * <p>
+     * The instance is changed here, in order to not change the
+     * instance the {@link #add(SparseGradient)} method should
+     * be used.
+     * </p>
+     * @param a instance to multiply
+     */
+    public void multInPlace(final SparseGradient a) {
                   ^^^^^^^^^^^

Shouldn't the name composed of full words (i.e be "multiplyInPlace")?

[...]
+
+    /**
+     * Get a hashCode for the derivative structure.
+     * @return a hash code value for this object
+     * @since 3.2
+     */
+    @Override
+    public int hashCode() {
+        return 743 + 809 *  + 233 * MathUtils.hash(value) + 167 *
                          ^^^^^^
Typo?

derivatives.hashCode();
+    }
+
+}


Regards,
Gilles


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to