You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gg...@apache.org on 2013/01/10 16:34:40 UTC
svn commit: r1431434 - in /logging/log4j/log4j2/trunk:
core/src/main/java/org/apache/logging/log4j/core/config/plugins/
core/src/main/java/org/apache/logging/log4j/core/net/
log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/
log4j12-api/src/test/ja...
Author: ggregory
Date: Thu Jan 10 15:34:39 2013
New Revision: 1431434
URL: http://svn.apache.org/viewvc?rev=1431434&view=rev
Log:
Remove unnecessary casts.
Modified:
logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java
logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java
logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java
Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java?rev=1431434&r1=1431433&r2=1431434&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java (original)
+++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java Thu Jan 10 15:34:39 2013
@@ -276,7 +276,7 @@ public class ResolverUtil<T> {
}
private void loadImplementationsInBundle(final Test test, final String packageName) {
- final BundleWiring wiring = (BundleWiring)FrameworkUtil.getBundle(ResolverUtil.class).adapt(BundleWiring.class);
+ final BundleWiring wiring = FrameworkUtil.getBundle(ResolverUtil.class).adapt(BundleWiring.class);
final Collection<String> list = wiring.listResources(packageName, "*.class", BundleWiring.LISTRESOURCES_RECURSE);
for (final String name : list) {
addIfMatching(test, name);
Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java?rev=1431434&r1=1431433&r2=1431434&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java (original)
+++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java Thu Jan 10 15:34:39 2013
@@ -117,7 +117,7 @@ public class SMTPManager extends Abstrac
String name = "SMTP:" + NameUtil.md5(sb.toString());
- return (SMTPManager) getManager(name, factory, new FactoryData(to, cc, bcc, from, replyTo, subject,
+ return getManager(name, factory, new FactoryData(to, cc, bcc, from, replyTo, subject,
protocol, host, port, username, password, isDebug, numElements));
}
Modified: logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java?rev=1431434&r1=1431433&r2=1431434&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java (original)
+++ logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java Thu Jan 10 15:34:39 2013
@@ -133,10 +133,10 @@ public class LoggerTest {
}
private Logger testMessageFactoryMismatch(final String name, final MessageFactory messageFactory1, final MessageFactory messageFactory2) {
- final Logger testLogger = (Logger) LogManager.getLogger(name, messageFactory1);
+ final Logger testLogger = LogManager.getLogger(name, messageFactory1);
assertNotNull(testLogger);
assertEquals(messageFactory1, testLogger.getMessageFactory());
- final Logger testLogger2 = (Logger) LogManager.getLogger(name, messageFactory2);
+ final Logger testLogger2 = LogManager.getLogger(name, messageFactory2);
assertEquals(messageFactory1, testLogger2.getMessageFactory());
return testLogger;
}
Modified: logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java
URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java?rev=1431434&r1=1431433&r2=1431434&view=diff
==============================================================================
--- logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java (original)
+++ logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java Thu Jan 10 15:34:39 2013
@@ -406,7 +406,7 @@ public class LoggerTest {
final List<LogEvent> msgs = appender.getEvents();
assertEquals(1, msgs.size());
- final LogEvent event = (LogEvent) msgs.get(0);
+ final LogEvent event = msgs.get(0);
assertEquals(org.apache.logging.log4j.Level.TRACE, event.getLevel());
assertEquals("Message 1", event.getMessage().getFormat());
appender.stop();
Re: svn commit: r1431434 - in /logging/log4j/log4j2/trunk:
core/src/main/java/org/apache/logging/log4j/core/config/plugins/
core/src/main/java/org/apache/logging/log4j/core/net/ log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/
log4j12-api/src/test/ja...
Posted by Gary Gregory <ga...@gmail.com>.
Thank you for fixing this again and exercising kindness! :)
Gary
On Thu, Jan 10, 2013 at 1:23 PM, Gary Gregory <ga...@gmail.com>wrote:
> Crud!!!!
>
> Gary
>
> On Jan 10, 2013, at 12:33, Ralph Goers <ra...@dslextreme.com> wrote:
>
> > I believe you made this first change before and it had to be reverted as
> it fails when compiling with Java 7.
> >
> > Ralph
> >
> >
> > On Jan 10, 2013, at 7:34 AM, ggregory@apache.org wrote:
> >
> >> Author: ggregory
> >> Date: Thu Jan 10 15:34:39 2013
> >> New Revision: 1431434
> >>
> >> URL: http://svn.apache.org/viewvc?rev=1431434&view=rev
> >> Log:
> >> Remove unnecessary casts.
> >>
> >> Modified:
> >>
> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
> >>
> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java
> >>
> logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java
> >>
> logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java
> >>
> >> Modified:
> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
> >> URL:
> http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java?rev=1431434&r1=1431433&r2=1431434&view=diff
> >>
> ==============================================================================
> >> ---
> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
> (original)
> >> +++
> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
> Thu Jan 10 15:34:39 2013
> >> @@ -276,7 +276,7 @@ public class ResolverUtil<T> {
> >> }
> >>
> >> private void loadImplementationsInBundle(final Test test, final
> String packageName) {
> >> - final BundleWiring wiring =
> (BundleWiring)FrameworkUtil.getBundle(ResolverUtil.class).adapt(BundleWiring.class);
> >> + final BundleWiring wiring =
> FrameworkUtil.getBundle(ResolverUtil.class).adapt(BundleWiring.class);
> >> final Collection<String> list =
> wiring.listResources(packageName, "*.class",
> BundleWiring.LISTRESOURCES_RECURSE);
> >> for (final String name : list) {
> >> addIfMatching(test, name);
> >>
> >> Modified:
> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java
> >> URL:
> http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java?rev=1431434&r1=1431433&r2=1431434&view=diff
> >>
> ==============================================================================
> >> ---
> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java
> (original)
> >> +++
> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java
> Thu Jan 10 15:34:39 2013
> >> @@ -117,7 +117,7 @@ public class SMTPManager extends Abstrac
> >>
> >> String name = "SMTP:" + NameUtil.md5(sb.toString());
> >>
> >> - return (SMTPManager) getManager(name, factory, new
> FactoryData(to, cc, bcc, from, replyTo, subject,
> >> + return getManager(name, factory, new FactoryData(to, cc, bcc,
> from, replyTo, subject,
> >> protocol, host, port, username, password, isDebug,
> numElements));
> >> }
> >>
> >>
> >> Modified:
> logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java
> >> URL:
> http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java?rev=1431434&r1=1431433&r2=1431434&view=diff
> >>
> ==============================================================================
> >> ---
> logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java
> (original)
> >> +++
> logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java
> Thu Jan 10 15:34:39 2013
> >> @@ -133,10 +133,10 @@ public class LoggerTest {
> >> }
> >>
> >> private Logger testMessageFactoryMismatch(final String name, final
> MessageFactory messageFactory1, final MessageFactory messageFactory2) {
> >> - final Logger testLogger = (Logger) LogManager.getLogger(name,
> messageFactory1);
> >> + final Logger testLogger = LogManager.getLogger(name,
> messageFactory1);
> >> assertNotNull(testLogger);
> >> assertEquals(messageFactory1, testLogger.getMessageFactory());
> >> - final Logger testLogger2 = (Logger) LogManager.getLogger(name,
> messageFactory2);
> >> + final Logger testLogger2 = LogManager.getLogger(name,
> messageFactory2);
> >> assertEquals(messageFactory1, testLogger2.getMessageFactory());
> >> return testLogger;
> >> }
> >>
> >> Modified:
> logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java
> >> URL:
> http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java?rev=1431434&r1=1431433&r2=1431434&view=diff
> >>
> ==============================================================================
> >> ---
> logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java
> (original)
> >> +++
> logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java
> Thu Jan 10 15:34:39 2013
> >> @@ -406,7 +406,7 @@ public class LoggerTest {
> >>
> >> final List<LogEvent> msgs = appender.getEvents();
> >> assertEquals(1, msgs.size());
> >> - final LogEvent event = (LogEvent) msgs.get(0);
> >> + final LogEvent event = msgs.get(0);
> >> assertEquals(org.apache.logging.log4j.Level.TRACE,
> event.getLevel());
> >> assertEquals("Message 1", event.getMessage().getFormat());
> >> appender.stop();
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
> > For additional commands, e-mail: log4j-dev-help@logging.apache.org
> >
>
--
E-Mail: garydgregory@gmail.com | ggregory@apache.org
JUnit in Action, 2nd Ed: <http://goog_1249600977>http://bit.ly/ECvg0
Spring Batch in Action: <http://s.apache.org/HOq>http://bit.ly/bqpbCK
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory
Re: svn commit: r1431434 - in /logging/log4j/log4j2/trunk:
core/src/main/java/org/apache/logging/log4j/core/config/plugins/
core/src/main/java/org/apache/logging/log4j/core/net/ log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/
log4j12-api/src/test/ja...
Posted by Gary Gregory <ga...@gmail.com>.
Crud!!!!
Gary
On Jan 10, 2013, at 12:33, Ralph Goers <ra...@dslextreme.com> wrote:
> I believe you made this first change before and it had to be reverted as it fails when compiling with Java 7.
>
> Ralph
>
>
> On Jan 10, 2013, at 7:34 AM, ggregory@apache.org wrote:
>
>> Author: ggregory
>> Date: Thu Jan 10 15:34:39 2013
>> New Revision: 1431434
>>
>> URL: http://svn.apache.org/viewvc?rev=1431434&view=rev
>> Log:
>> Remove unnecessary casts.
>>
>> Modified:
>> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
>> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java
>> logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java
>> logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java
>>
>> Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
>> URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java?rev=1431434&r1=1431433&r2=1431434&view=diff
>> ==============================================================================
>> --- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java (original)
>> +++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java Thu Jan 10 15:34:39 2013
>> @@ -276,7 +276,7 @@ public class ResolverUtil<T> {
>> }
>>
>> private void loadImplementationsInBundle(final Test test, final String packageName) {
>> - final BundleWiring wiring = (BundleWiring)FrameworkUtil.getBundle(ResolverUtil.class).adapt(BundleWiring.class);
>> + final BundleWiring wiring = FrameworkUtil.getBundle(ResolverUtil.class).adapt(BundleWiring.class);
>> final Collection<String> list = wiring.listResources(packageName, "*.class", BundleWiring.LISTRESOURCES_RECURSE);
>> for (final String name : list) {
>> addIfMatching(test, name);
>>
>> Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java
>> URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java?rev=1431434&r1=1431433&r2=1431434&view=diff
>> ==============================================================================
>> --- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java (original)
>> +++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java Thu Jan 10 15:34:39 2013
>> @@ -117,7 +117,7 @@ public class SMTPManager extends Abstrac
>>
>> String name = "SMTP:" + NameUtil.md5(sb.toString());
>>
>> - return (SMTPManager) getManager(name, factory, new FactoryData(to, cc, bcc, from, replyTo, subject,
>> + return getManager(name, factory, new FactoryData(to, cc, bcc, from, replyTo, subject,
>> protocol, host, port, username, password, isDebug, numElements));
>> }
>>
>>
>> Modified: logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java
>> URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java?rev=1431434&r1=1431433&r2=1431434&view=diff
>> ==============================================================================
>> --- logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java (original)
>> +++ logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java Thu Jan 10 15:34:39 2013
>> @@ -133,10 +133,10 @@ public class LoggerTest {
>> }
>>
>> private Logger testMessageFactoryMismatch(final String name, final MessageFactory messageFactory1, final MessageFactory messageFactory2) {
>> - final Logger testLogger = (Logger) LogManager.getLogger(name, messageFactory1);
>> + final Logger testLogger = LogManager.getLogger(name, messageFactory1);
>> assertNotNull(testLogger);
>> assertEquals(messageFactory1, testLogger.getMessageFactory());
>> - final Logger testLogger2 = (Logger) LogManager.getLogger(name, messageFactory2);
>> + final Logger testLogger2 = LogManager.getLogger(name, messageFactory2);
>> assertEquals(messageFactory1, testLogger2.getMessageFactory());
>> return testLogger;
>> }
>>
>> Modified: logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java
>> URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java?rev=1431434&r1=1431433&r2=1431434&view=diff
>> ==============================================================================
>> --- logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java (original)
>> +++ logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java Thu Jan 10 15:34:39 2013
>> @@ -406,7 +406,7 @@ public class LoggerTest {
>>
>> final List<LogEvent> msgs = appender.getEvents();
>> assertEquals(1, msgs.size());
>> - final LogEvent event = (LogEvent) msgs.get(0);
>> + final LogEvent event = msgs.get(0);
>> assertEquals(org.apache.logging.log4j.Level.TRACE, event.getLevel());
>> assertEquals("Message 1", event.getMessage().getFormat());
>> appender.stop();
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
> For additional commands, e-mail: log4j-dev-help@logging.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org
Re: svn commit: r1431434 - in /logging/log4j/log4j2/trunk: core/src/main/java/org/apache/logging/log4j/core/config/plugins/ core/src/main/java/org/apache/logging/log4j/core/net/ log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/ log4j12-api/src/test/ja...
Posted by Ralph Goers <ra...@dslextreme.com>.
I took care of this. I put a comment in to remind you the next time you want to "fix" this ;-)
Ralph
On Jan 10, 2013, at 9:32 AM, Ralph Goers wrote:
> I believe you made this first change before and it had to be reverted as it fails when compiling with Java 7.
>
> Ralph
>
>
> On Jan 10, 2013, at 7:34 AM, ggregory@apache.org wrote:
>
>> Author: ggregory
>> Date: Thu Jan 10 15:34:39 2013
>> New Revision: 1431434
>>
>> URL: http://svn.apache.org/viewvc?rev=1431434&view=rev
>> Log:
>> Remove unnecessary casts.
>>
>> Modified:
>> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
>> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java
>> logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java
>> logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java
>>
>> Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
>> URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java?rev=1431434&r1=1431433&r2=1431434&view=diff
>> ==============================================================================
>> --- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java (original)
>> +++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java Thu Jan 10 15:34:39 2013
>> @@ -276,7 +276,7 @@ public class ResolverUtil<T> {
>> }
>>
>> private void loadImplementationsInBundle(final Test test, final String packageName) {
>> - final BundleWiring wiring = (BundleWiring)FrameworkUtil.getBundle(ResolverUtil.class).adapt(BundleWiring.class);
>> + final BundleWiring wiring = FrameworkUtil.getBundle(ResolverUtil.class).adapt(BundleWiring.class);
>> final Collection<String> list = wiring.listResources(packageName, "*.class", BundleWiring.LISTRESOURCES_RECURSE);
>> for (final String name : list) {
>> addIfMatching(test, name);
>>
>> Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java
>> URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java?rev=1431434&r1=1431433&r2=1431434&view=diff
>> ==============================================================================
>> --- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java (original)
>> +++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java Thu Jan 10 15:34:39 2013
>> @@ -117,7 +117,7 @@ public class SMTPManager extends Abstrac
>>
>> String name = "SMTP:" + NameUtil.md5(sb.toString());
>>
>> - return (SMTPManager) getManager(name, factory, new FactoryData(to, cc, bcc, from, replyTo, subject,
>> + return getManager(name, factory, new FactoryData(to, cc, bcc, from, replyTo, subject,
>> protocol, host, port, username, password, isDebug, numElements));
>> }
>>
>>
>> Modified: logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java
>> URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java?rev=1431434&r1=1431433&r2=1431434&view=diff
>> ==============================================================================
>> --- logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java (original)
>> +++ logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java Thu Jan 10 15:34:39 2013
>> @@ -133,10 +133,10 @@ public class LoggerTest {
>> }
>>
>> private Logger testMessageFactoryMismatch(final String name, final MessageFactory messageFactory1, final MessageFactory messageFactory2) {
>> - final Logger testLogger = (Logger) LogManager.getLogger(name, messageFactory1);
>> + final Logger testLogger = LogManager.getLogger(name, messageFactory1);
>> assertNotNull(testLogger);
>> assertEquals(messageFactory1, testLogger.getMessageFactory());
>> - final Logger testLogger2 = (Logger) LogManager.getLogger(name, messageFactory2);
>> + final Logger testLogger2 = LogManager.getLogger(name, messageFactory2);
>> assertEquals(messageFactory1, testLogger2.getMessageFactory());
>> return testLogger;
>> }
>>
>> Modified: logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java
>> URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java?rev=1431434&r1=1431433&r2=1431434&view=diff
>> ==============================================================================
>> --- logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java (original)
>> +++ logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java Thu Jan 10 15:34:39 2013
>> @@ -406,7 +406,7 @@ public class LoggerTest {
>>
>> final List<LogEvent> msgs = appender.getEvents();
>> assertEquals(1, msgs.size());
>> - final LogEvent event = (LogEvent) msgs.get(0);
>> + final LogEvent event = msgs.get(0);
>> assertEquals(org.apache.logging.log4j.Level.TRACE, event.getLevel());
>> assertEquals("Message 1", event.getMessage().getFormat());
>> appender.stop();
>>
>>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org
Re: svn commit: r1431434 - in /logging/log4j/log4j2/trunk: core/src/main/java/org/apache/logging/log4j/core/config/plugins/ core/src/main/java/org/apache/logging/log4j/core/net/ log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/ log4j12-api/src/test/ja...
Posted by Ralph Goers <ra...@dslextreme.com>.
I believe you made this first change before and it had to be reverted as it fails when compiling with Java 7.
Ralph
On Jan 10, 2013, at 7:34 AM, ggregory@apache.org wrote:
> Author: ggregory
> Date: Thu Jan 10 15:34:39 2013
> New Revision: 1431434
>
> URL: http://svn.apache.org/viewvc?rev=1431434&view=rev
> Log:
> Remove unnecessary casts.
>
> Modified:
> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
> logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java
> logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java
> logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java
>
> Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java
> URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java?rev=1431434&r1=1431433&r2=1431434&view=diff
> ==============================================================================
> --- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java (original)
> +++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/config/plugins/ResolverUtil.java Thu Jan 10 15:34:39 2013
> @@ -276,7 +276,7 @@ public class ResolverUtil<T> {
> }
>
> private void loadImplementationsInBundle(final Test test, final String packageName) {
> - final BundleWiring wiring = (BundleWiring)FrameworkUtil.getBundle(ResolverUtil.class).adapt(BundleWiring.class);
> + final BundleWiring wiring = FrameworkUtil.getBundle(ResolverUtil.class).adapt(BundleWiring.class);
> final Collection<String> list = wiring.listResources(packageName, "*.class", BundleWiring.LISTRESOURCES_RECURSE);
> for (final String name : list) {
> addIfMatching(test, name);
>
> Modified: logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java
> URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java?rev=1431434&r1=1431433&r2=1431434&view=diff
> ==============================================================================
> --- logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java (original)
> +++ logging/log4j/log4j2/trunk/core/src/main/java/org/apache/logging/log4j/core/net/SMTPManager.java Thu Jan 10 15:34:39 2013
> @@ -117,7 +117,7 @@ public class SMTPManager extends Abstrac
>
> String name = "SMTP:" + NameUtil.md5(sb.toString());
>
> - return (SMTPManager) getManager(name, factory, new FactoryData(to, cc, bcc, from, replyTo, subject,
> + return getManager(name, factory, new FactoryData(to, cc, bcc, from, replyTo, subject,
> protocol, host, port, username, password, isDebug, numElements));
> }
>
>
> Modified: logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java
> URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java?rev=1431434&r1=1431433&r2=1431434&view=diff
> ==============================================================================
> --- logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java (original)
> +++ logging/log4j/log4j2/trunk/log4j-to-slf4j/src/test/java/org/apache/logging/slf4j/LoggerTest.java Thu Jan 10 15:34:39 2013
> @@ -133,10 +133,10 @@ public class LoggerTest {
> }
>
> private Logger testMessageFactoryMismatch(final String name, final MessageFactory messageFactory1, final MessageFactory messageFactory2) {
> - final Logger testLogger = (Logger) LogManager.getLogger(name, messageFactory1);
> + final Logger testLogger = LogManager.getLogger(name, messageFactory1);
> assertNotNull(testLogger);
> assertEquals(messageFactory1, testLogger.getMessageFactory());
> - final Logger testLogger2 = (Logger) LogManager.getLogger(name, messageFactory2);
> + final Logger testLogger2 = LogManager.getLogger(name, messageFactory2);
> assertEquals(messageFactory1, testLogger2.getMessageFactory());
> return testLogger;
> }
>
> Modified: logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java
> URL: http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java?rev=1431434&r1=1431433&r2=1431434&view=diff
> ==============================================================================
> --- logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java (original)
> +++ logging/log4j/log4j2/trunk/log4j12-api/src/test/java/org/apache/log4j/LoggerTest.java Thu Jan 10 15:34:39 2013
> @@ -406,7 +406,7 @@ public class LoggerTest {
>
> final List<LogEvent> msgs = appender.getEvents();
> assertEquals(1, msgs.size());
> - final LogEvent event = (LogEvent) msgs.get(0);
> + final LogEvent event = msgs.get(0);
> assertEquals(org.apache.logging.log4j.Level.TRACE, event.getLevel());
> assertEquals("Message 1", event.getMessage().getFormat());
> appender.stop();
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: log4j-dev-unsubscribe@logging.apache.org
For additional commands, e-mail: log4j-dev-help@logging.apache.org