Uploaded image for project: 'Log4cxx'
  1. Log4cxx
  2. LOGCXX-236

Re-order constructor initialiser lists to match declaration order

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.10.0
    • Fix Version/s: 0.10.0
    • Component/s: None
    • Labels:
      None

      Description

      The patch I'm about to attach fixes warnings with any version of GCC, caused by
      initialising members in the wrong order in a constructor initialiser
      list. The standard says that members will be initialised in the order
      they're declared in the class body, not the order they appear in the
      init list, so for clarity's sake the init list should be in the same
      order.

        Activity

        Hide
        carnold@apache.org Curt Arnold added a comment -

        Committed rev 627216.

        Show
        carnold@apache.org Curt Arnold added a comment - Committed rev 627216.

          People

          • Assignee:
            carnold@apache.org Curt Arnold
            Reporter:
            jwakely Jonathan Wakely
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development