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

Put typedefs into log4cxx namespace

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 1.0.0
    • None
    • None

    Description

      We have a handful of typedefs that are defined in the global namespace.  It would probably make sense to:

      1. put relevant typedefs inside the log4cxx namespace
      2. remove typedefs that aren't needed.  For example, the log4cxx_int64_t and log4cxx_uint32_t should be able to be replaced by standard ones in <cstdint>.

      It is possible that we could be on a system that does not support uint32_t(for example), but that seems very unlikely to be the case.  In the worst case, we can always add a test to the cmake configuration to see if we have these constants and change things appropriately.

      Attachments

        Issue Links

          Activity

            People

              rmiddleton Robert Middleton
              rmiddleton Robert Middleton
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: