Bug 46512 - CategoryPath doesn't replace slashes to dots
CategoryPath doesn't replace slashes to dots
Status: RESOLVED FIXED
Product: Log4j
Classification: Unclassified
Component: Other
1.2
PC Linux
: P2 minor
: ---
Assigned To: log4j-dev
:
Depends on:
Blocks:
  Show dependency tree
 
Reported: 2009-01-12 06:07 UTC by Przemyslaw Wesolek
Modified: 2009-01-13 12:23 UTC (History)
0 users



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Przemyslaw Wesolek 2009-01-12 06:07:18 UTC
Running FindBugs on log4j sources -- as part of preparing lecture, in fact -- revealed a bug in org.apache.log4j.lf5.viewer.categoryexplorer.CategoryPath.

In line 64 there is a code

  processedCategory.replace('/', '.');

and FindBugs correctly recognizes the error there as the result of replacement is ignored. The line should be

  processedCategory = processedCategory.replace('/', '.');
Comment 1 Curt Arnold 2009-01-13 12:23:39 UTC
LogFactor5 is long deprecated and is no longer maintained.  Committed change in rev 734230.