Uploaded image for project: 'Commons Net'
  1. Commons Net
  2. NET-621

SubnetUtils#SubnetInfo - remove unnecessary accessors

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 3.7
    • None
    • None

    Description

      The below methods do not actually work.

      • private int netmask() { return netmask; }
      • private int network() { return network; }
      • private int address() { return address; }
      • private int broadcast() { return broadcast; }

      So, I suggest to remove them.
      https://github.com/apache/commons-net/pull/8

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            Umoxfo Makoto Sakaguchi
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment