hg: nashorn/jdk8/nashorn: 8005801: Refactor findSetMethod
Marcus Lagergren
marcus.lagergren at oracle.com
Tue Jan 8 05:52:09 PST 2013
+1
On Jan 8, 2013, at 2:14 PM, attila.szegedi at oracle.com wrote:
> Changeset: 4620ac94e7dc
> Author: attila
> Date: 2013-01-08 14:14 +0100
> URL: http://hg.openjdk.java.net/nashorn/jdk8/nashorn/rev/4620ac94e7dc
>
> 8005801: Refactor findSetMethod
> Summary: findSetMethod() was a very large single method, very unreadable and unmaintainable. It was broken into easy-to-understand pieces. The refactoring required introduction of a comand-object like entity, SetMethodCreator, to contain the nontrivial transient state of the algorithm that made the original big method so resistant to refactoring in the first place.
> Reviewed-by: lagergren, sundar
>
> ! src/jdk/nashorn/internal/runtime/ScriptObject.java
> + src/jdk/nashorn/internal/runtime/SetMethodCreator.java
> ! src/jdk/nashorn/internal/runtime/SpillProperty.java
> - src/jdk/nashorn/internal/runtime/linker/Mangler.java
>
More information about the nashorn-dev
mailing list