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

Surround calls to log() function with an if statement to avoid costly string concats if the logger is not enabled

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.0.5
    • Fix Version/s: 1.1.0
    • Component/s: Examples
    • Labels:
      None

      Description

      This is an obviously low-priority RFE

      Would be a nice idea to surround all calls to the logging code with
      if(logger.isDebugEnabled(this)) or logger.isInfoEnabled() to avoid the cost of string concats in the case of loggers actually being filtered out.

      this is probably only applicable to all the incoming/outgoing FIX message logging, may not be that expensive but probably a good idea.

        Attachments

          Activity

            People

            • Assignee:
              admin Steve Bate
              Reporter:
              toli Toli Kuznets
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: