Bug 46512 - CategoryPath doesn't replace slashes to dots
Summary: CategoryPath doesn't replace slashes to dots
Status: RESOLVED FIXED
Alias: None
Product: Log4j - Now in Jira
Classification: Unclassified
Component: Other (show other bugs)
Version: 1.2
Hardware: PC Linux
: P2 minor
Target Milestone: ---
Assignee: log4j-dev
URL:
Keywords:
Depends on:
Blocks:
 
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.