Log4cxx
  1. Log4cxx
  2. LOGCXX-31

Missing const qualifiers, Exception::getMessage() in particular.

    Details

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

      Description

      Conway Allen reported that Exception::getMessage() should be declared const in http://nagoya.apache.org/eyebrowse/ReadMsg?listName=log4cxx-user@logging.apache.org&msgNo=445

      I've broadened this bug and may keep it open for a while to fix other missing const's.

        Activity

        Hide
        Curt Arnold added a comment -

        Initial CVS commit: http://nagoya.apache.org/eyebrowse/ReadMsg?listName=log4cxx-dev@logging.apache.org&msgNo=323 (or nearby)

        Changed getMessage to getMessage() const and changed message member variable to const which triggered some other good but unexpected changes (eliminating default constructors on IOException and duplicate message members in derived exceptions).

        Show
        Curt Arnold added a comment - Initial CVS commit: http://nagoya.apache.org/eyebrowse/ReadMsg?listName=log4cxx-dev@logging.apache.org&msgNo=323 (or nearby) Changed getMessage to getMessage() const and changed message member variable to const which triggered some other good but unexpected changes (eliminating default constructors on IOException and duplicate message members in derived exceptions).

          People

          • Assignee:
            Curt Arnold
            Reporter:
            Curt Arnold
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development