Derby
  1. Derby
  2. DERBY-6063

Access static variables by class name in TabInfoImpl, InsertResultSet and ControlRow

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 10.10.1.1
    • Fix Version/s: 10.10.1.1
    • Component/s: SQL, Store
    • Labels:
      None

      Description

      Accessing static variables via an instance is causing trouble for a static code analysis application. Since this way of accessing static variables is not recommended, I'm changing the code to access the static variables via class names.

      I'm also removing an unused method in access/bree/ControlRow.

        Activity

        Hide
        Kristian Waagan added a comment -

        Attaching patch 1a.
        The patch changes the access for static variables from instances to class names in three Derby classes, removes an unused method in ControlRow, and removes some unused local variables I spotted when browsing these three classes.

        Patch ready for review.

        Show
        Kristian Waagan added a comment - Attaching patch 1a. The patch changes the access for static variables from instances to class names in three Derby classes, removes an unused method in ControlRow, and removes some unused local variables I spotted when browsing these three classes. Patch ready for review.
        Hide
        Kristian Waagan added a comment -

        suites.All completed without errors.

        Committed patch 1a to trunk with revision 1441396.
        Resolving issue.

        Show
        Kristian Waagan added a comment - suites.All completed without errors. Committed patch 1a to trunk with revision 1441396. Resolving issue.

          People

          • Assignee:
            Kristian Waagan
            Reporter:
            Kristian Waagan
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development