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 2006/09/01 06:48:12 UTC
DO NOT REPLY [Bug 34762] - Scheduler uses notify, not notifyAll
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=34762>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=34762
carnold@apache.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
Summary|Cleanup of source; remove |Scheduler uses notify, not
|unused variables, etc. |notifyAll
------- Additional Comments From carnold@apache.org 2006-09-01 04:48 -------
Some suggested changes had already been fixed or the underlying code eliminated or substantially
rewritten. Many of the changes were to variables that were not declared a private which means that
eliminating them may cause binary-compatibility problems though usually only limited to code that
inserted itself into a org.apache.log4j package to access package visible members.
The two changes that were committed were to make NA_LOCATION_INFO final and changing the notify's in
Scheduler to notifyAll and synchronizing the close method. Scheduler has been very problematic in the
Gump tests, use of notify might have been at least partially responsible. I'm changing the title to reflect
the major effect of the change.
Committed rev 439177.
--
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org