On Mon, May 4, 2009 at 4:53 PM, Iustin Pop <[email protected]> wrote: > > This patch fixes two issues with LUSetClusterParams and argument > checking. > > First, this LU used the wrong function name (CheckParameters instead of > CheckArguments), which means that no parameter checking was done at all; > this impacted the candidate_pool_size checks (the only one done at this > stage). > > Second, int() can raise both ValueError and TypeError, and we should > correctly handle both. >
LGTM Thanks, Guido
