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

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

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: 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
        carnold@apache.org 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
        carnold@apache.org 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:
            carnold@apache.org Curt Arnold
            Reporter:
            carnold@apache.org Curt Arnold
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development