[Ovirt-devel] [PATCH] Ruby interface for Cobbler XML-RPC APIs.

[Jason Guiditta]

On Fri, 2008-08-01 at 12:06 +0100, John Leach wrote:
> On Fri, 2008-08-01 at 02:21 +0000, David Lutterkort wrote:
> > Don't you want to cache the connection here, i.e. more something like
> > 
> >   def self.connection(writable = false)
> >      unless @@connection
> >        @@connection = XMLRPC::Client.new2("http://#{@@hostname}/cobbler_api#{writable ? '_rw' : ''}")
> >      end
> >       
> >      return @@connection
> >   end
> 
> Not to turn this into a Ruby style thread but you can do this, which is
> shorter, pretty common and actually faster (return is slow):
> 
> def self.connection(writable = false)
>   @@connections ||= XMLRPC::Client.new2("http://#{@@hostname}/cobbler_api#{writable ? '_rw' : ''}")
> end
> 
> John.
+1, not to mention that 'return' is just unneeded, since that is what
happens by default, so at the very least it makes the code cleaner