[MATH-822] Unnecessary Parameter in EigenDecomposition?

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.1
• Assignee: Unassigned
• Reporter: Jared Becksfort
• Created: 13/Jul/12 15:18
• Updated: 04/Mar/13 18:53
• Resolved: 14/Jul/12 21:19

Description

I am still new to the Commons Math code base, so I am sorry if this is a dumb question. I am browsing the code for EigenDecomposition, and both of its constructors have an unused parameter called splitTolerance. The comments indicate that this is for backward compatibility.

Would it be acceptable for me to simply include a one-parameter constructor that calls the two-parameter one, so backward compatibility is preserved and client code no longer needs to provide a pointless double?