Uploaded image for project: 'QuickFIX/J'
  1. QuickFIX/J
  2. QFJ-879

SystemTime currentTimeMillis is synchronized

          Details

          • Type: Bug
          • Status: Closed
          • Priority: Default
          • Resolution: Fixed
          • Affects Version/s: 1.6.0, 1.6.1
          • Fix Version/s: 1.6.2
          • Component/s: Engine
          • Labels:
            None

            Description

            The class SystemTime was created to wrap a system time source primarily for unit testing (according to its JavaDocs) but it ended up been used globally so many important classes are obtaining the current time in millis using a synchronized call for no reason.

            Can you please check if such thing is needed? If so let me know so that I can send a PR via Git.

              Attachments

                Activity

                  People

                  • Assignee:
                    oxyg3n8@gmail.com Guido Medina
                    Reporter:
                    oxyg3n8@gmail.com Guido Medina
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    2 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: