Did this change break the build?  Jenkins is showing test failures in 
TypeConverterRegistryTest.

Ralph

> Begin forwarded message:
> 
> From: [email protected]
> Subject: logging-log4j2 git commit: Make sure the value associated with the 
> key is returned.
> Date: June 27, 2015 at 12:40:08 AM MST
> To: [email protected]
> Reply-To: [email protected]
> 
> Repository: logging-log4j2
> Updated Branches:
>  refs/heads/master a7e04f804 -> 70b72ad6f
> 
> 
> Make sure the value associated with the key is returned.
> 
> Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
> Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/70b72ad6
> Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/70b72ad6
> Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/70b72ad6
> 
> Branch: refs/heads/master
> Commit: 70b72ad6fabf976feb386715fa27d1b6061ca704
> Parents: a7e04f8
> Author: ggregory <[email protected]>
> Authored: Sat Jun 27 00:40:06 2015 -0700
> Committer: ggregory <[email protected]>
> Committed: Sat Jun 27 00:40:06 2015 -0700
> 
> ----------------------------------------------------------------------
> .../log4j/core/config/plugins/convert/TypeConverterRegistry.java  | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
> ----------------------------------------------------------------------
> 
> 
> http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/70b72ad6/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/convert/TypeConverterRegistry.java
> ----------------------------------------------------------------------
> diff --git 
> a/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/convert/TypeConverterRegistry.java
>  
> b/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/convert/TypeConverterRegistry.java
> index d6e0670..12d364f 100644
> --- 
> a/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/convert/TypeConverterRegistry.java
> +++ 
> b/log4j-core/src/main/java/org/apache/logging/log4j/core/config/plugins/convert/TypeConverterRegistry.java
> @@ -96,8 +96,7 @@ public class TypeConverterRegistry {
>             if (TypeUtil.isAssignable(type, key)) {
>                 LOGGER.debug("Found compatible TypeConverter<{}> for type 
> [{}].", key, type);
>                 final TypeConverter<?> value = entry.getValue();
> -                registry.putIfAbsent(type, value);
> -                return value;
> +                return registry.putIfAbsent(type, value);
>             }
>         }
>         throw new UnknownFormatConversionException(type.toString());
> 

Reply via email to