Re: Extensions to ipaddr.rb

> Is this your intention?
> 
>   puts IPAddr.new("192.168.0.0") + "1"          # => 192.168.0.1
> 
> I think you should be using #to_int, not #to_i.

Two points in response to that:

(1) The existing operator methods, like & and |, use to_i. For example:

  ...
  # Returns a new ipaddr built by bitwise AND.
  def &(other)
    return self.clone.set(@addr & other.to_i)
  end
  ...

I think it's better to be consistent with what's already there.

(2) The main reason for to_i on the RHS is to be able to do things like

    a = IPAddr.new("192.168.0.123")
    b = IPAddr.new("255.255.255.240")
    c = a & b

However the existing IPAddr class doesn't implement to_int:

irb(main):001:0> require 'ipaddr'
=> true
irb(main):002:0> IPAddr.new("192.168.0.0").to_int
NoMethodError: undefined method `to_int' for #<IPAddr: IPv4:192.168.0.0/255.255.255.255>
        from (irb):2

Therefore, if someone decided to change all these operators to call to_int
instead of to_i, then they would have to add a to_int method to IPAddr as
well.

I'd consider this sort of 'tidying up' as an orthogonal issue to the
extensions I'm proposing, and I'd hope one wouldn't delay the other.

Regards,

Brian.