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

Hi,

In <ae6cb1100711272327q24872418j9339042d7377ceb2@mail.gmail.com>
  "Re: svn commit: r28018 - trunk/subversion/bindings/swig/ruby/svn" on Tue, 27 Nov 2007 23:27:43 -0800,
  "Joe Swatosh" <joe.swatosh@gmail.com> wrote:

> > 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.

Thanks!

--
kou
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org