From: Ville V. <vil...@gm...> - 2008-09-20 11:10:34
|
Yet again, no regressions and mostly deletes code. This patch adds a vararg helper for the required parameter binding. I'm going to take a break of Closure.java for now. :) Some minor things that could be done are 1) using minArgs instead of 1-8 in the bindParameters invocations (bad magic numbers) 2) the else blocks with their array construction could be cleaned up. Either by constructing the array with the values directly, instead of setting each array member - or adding a vararg function that calls execute. Then call this vararg function from the else blocks. |