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

Hi

On Nov 28, 2007 3:45 AM, Kouhei Sutou <kou@cozmixng.org> wrote:
> 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!

No problem. With the talk of an eminent branch for 1.5, should we try to
add this type of interface anywhere else? I don't believe there would be
time to do it everywhere, but maybe we should come up with a prioritized
list? My first thought is the client, after that....?

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