Project

General

Profile

Actions

Feature #11700

open

positive flags for reverse lookup to socket

Added by nobu (Nobuyoshi Nakada) over 8 years ago. Updated over 8 years ago.

Status:
Open
Assignee:
-
Target version:
-
[ruby-core:71511]

Description

Now BasicSocket has do_not_reverse_lookup global and per-socket attributes.
Negative flag makes less readability.
So I propose the inverse and positive attributes,

  • BasicSocket.reverse_lookup
  • BasicSocket.reverse_lookup=
  • BasicSocket#reverse_lookup
  • BasicSocket#reverse_lookup=

These methods share the same flags with do_not_reverse_lookup, just invert input/output values.

patch

Updated by normalperson (Eric Wong) over 8 years ago

wrote:

Now BasicSocket has do_not_reverse_lookup global and per-socket attributes.
Negative flag makes less readability.
So I propose the inverse and positive attributes,

  • BasicSocket.reverse_lookup
  • BasicSocket.reverse_lookup=

I prefer we skip these globals and deprecate the existing global ones.
(and maybe even deprecating per-socket).

It's a complete performance killer to enable reverse lookup and nobody
should be doing it for every socket.

Actions

Also available in: Atom PDF

Like0
Like0