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

Article.references field is never updated; extremely inefficient implementation

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.0
    • 3.0
    • NNTP
    • None

    Description

      Article.references is a private StringBuffer instance.

      The method addReference(String) inserts the reference into the buffer followed by a tab.
      However, as far as I can tell, this is never called.

      The method getReferences() - which is used - has to parse the StringBuffer in order to find the references.
      Since the buffer is only used in the add and get methods, it does not make sense to handle the references this way.
      Even if the buffer were used (e.g. as part of a toString() method) it would still be better to store the references as a List or Array.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              sebb Sebb
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: