[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r28018 - trunk/subversion/bindings/swig/ruby/svn

From: Joe Swatosh <joe.swatosh_at_gmail.com>
Date: 2007-11-28 08:27:43 CET

Hi kou,

On Nov 26, 2007 2:39 AM, Kouhei Sutou <kou@cozmixng.org> wrote:
> Hi,
>

>
> What about that we change all default argument values of
> update_editor to nil and remove nil value argument from
> update_editor2 arguments?
>
> def update_editor(xxx, yyy=nil, zzz=nil, ...)
> update_editor2(:xxx => xxx, :yyy => yyy, :zzz => zzz, ...)
> end
>
> def update_editor2(arguments={})
> ...
> arguments = arguments.reject {|k, v| v.nil?}
> arguments = optional_arguments_defaults.merge(arguments)
> ...
> end
>

Good idea. Commit r28111.

--
Joe
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Nov 28 08:27:53 2007

This is an archived mail posted to the Subversion Dev mailing list.