You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4j-dev@logging.apache.org by bu...@apache.org on 2012/09/29 20:53:23 UTC

[Bug 53948] New: ThrowableInformationPatternConverter does not need to import ThrowableInformation

https://issues.apache.org/bugzilla/show_bug.cgi?id=53948

          Priority: P2
            Bug ID: 53948
          Assignee: log4j-dev@logging.apache.org
           Summary: ThrowableInformationPatternConverter does not need to
                    import ThrowableInformation
          Severity: enhancement
    Classification: Unclassified
                OS: All
          Reporter: harsh@cloudera.com
          Hardware: All
            Status: NEW
           Version: 1.2
         Component: Other
           Product: Log4j

Trivial/Non-problem/Code-nit thing:

Since LoggingEvent.getThrowableStrRep(); exists, we do not need to use and
store the LoggingEvent.getThrowableInformation() returned object for the
String[] representation of the throwable information.

A patch is attached that refactors this, leading to one less import.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org

[Bug 53948] ThrowableInformationPatternConverter does not need to import ThrowableInformation

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=53948

Harsh J <ha...@cloudera.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P2                          |P5

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org

[Bug 53948] ThrowableInformationPatternConverter does not need to import ThrowableInformation

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=53948

Harsh J <ha...@cloudera.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |harsh@cloudera.com

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org

[Bug 53948] ThrowableInformationPatternConverter does not need to import ThrowableInformation

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=53948

--- Comment #1 from Harsh J <ha...@cloudera.com> ---
Created attachment 29429
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=29429&action=edit
Patch that refactors the method to not require an extra import

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org