String#split seems to be faster than capturing digits with Regexp

ruby · Jan 6, 2023 · e0feb07 · e0feb07
1 parent d92acb3
commit e0feb07
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/ipaddr.rb b/lib/ipaddr.rb
@@ -52,7 +52,7 @@ class IPAddr
   # Regexp _internally_ used for parsing IPv4 address.
   RE_IPV4ADDRLIKE = %r{
     \A
-    (\d+) \. (\d+) \. (\d+) \. (\d+)
+    \d+ \. \d+ \. \d+ \. \d+
     \z
   }x
 
@@ -647,8 +647,8 @@ def in_addr(addr)
     when Array
       octets = addr
     else
-      m = RE_IPV4ADDRLIKE.match(addr) or return nil
-      octets = m.captures
+      RE_IPV4ADDRLIKE.match?(addr) or return nil
+      octets = addr.split('.')
     end
     octets.inject(0) { |i, s|
       (n = s.to_i) < 256 or raise InvalidAddressError, "invalid address: #{@addr}"