Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-11004

Add missing ‘synchronized’ modifier

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • Trunk
    • 16.11.06, 17.12.01, 18.12.01
    • framework
    • None

    Description

      Overidding a synchronized method with a non-synchronized one can
      introduce potential runtime concurrency bugs. Since
      ‘ExtendedProperties#loadFromXML’ is overridding
      ‘Propertiers#loadFromXML’ which is synchronized, it is safer to mark
      it as synchonized.

      Attachments

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

          People

            mthl Mathieu Lirzin (Inactive)
            mthl Mathieu Lirzin (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment