Derby
  1. Derby
  2. DERBY-5491

[patch] new String("") or new String() is pointless, and just allocates memory needlessly - removed

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 10.9.1.0
    • Component/s: Miscellaneous
    • Labels:
      None
    • Urgency:
      Low

      Description

      new String("::") is pointless, use "::"
      same with new String() vs ""

        Activity

        Dave Brosius created issue -
        Dave Brosius made changes -
        Field Original Value New Value
        Attachment no_new_string.diff [ 12502643 ]
        Hide
        Knut Anders Hatlen added a comment -

        Thanks, Dave. The patch looks good to me, and all the regression tests passed.

        Committed revision 1198653.

        Show
        Knut Anders Hatlen added a comment - Thanks, Dave. The patch looks good to me, and all the regression tests passed. Committed revision 1198653.
        Knut Anders Hatlen made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Issue & fix info Patch Available [ 10102 ]
        Assignee Dave Brosius [ dbrosius@apache.org ]
        Fix Version/s 10.9.0.0 [ 12316344 ]
        Resolution Fixed [ 1 ]
        Knut Anders Hatlen made changes -
        Component/s Miscellaneous [ 11400 ]
        Hide
        Knut Anders Hatlen added a comment -

        [bulk update] Close all resolved issues that haven't been updated for more than one year.

        Show
        Knut Anders Hatlen added a comment - [bulk update] Close all resolved issues that haven't been updated for more than one year.
        Knut Anders Hatlen made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Gavin made changes -
        Workflow jira [ 12641188 ] Default workflow, editable Closed status [ 12796757 ]

          People

          • Assignee:
            Dave Brosius
            Reporter:
            Dave Brosius
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development