Can we add entries in the change log for deprecated methods or constants that 
were removed? 

Otherwise the only way to tell for users is when they get a compilation 
error... 

Sent from my iPhone

> On 2016/08/23, at 14:54, [email protected] wrote:
> 
> Repository: logging-log4j2
> Updated Branches:
>  refs/heads/master 99c96e705 -> facb281ba
> 
> 
> Remove internal method that is @deprecated Use {@link
> #createLogger(boolean, Level, String, String, AppenderRef[], Property[],
> Configuration, Filter)}
> 
> Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
> Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/facb281b
> Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/facb281b
> Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/facb281b
> 
> Branch: refs/heads/master
> Commit: facb281ba2f5f83903a7aedf385f83d7f9520561
> Parents: 99c96e7
> Author: Gary Gregory <[email protected]>
> Authored: Mon Aug 22 22:54:57 2016 -0700
> Committer: Gary Gregory <[email protected]>
> Committed: Mon Aug 22 22:54:57 2016 -0700
> 
> ----------------------------------------------------------------------
> .../logging/log4j/core/config/LoggerConfig.java | 34 --------------------
> 1 file changed, 34 deletions(-)
> ----------------------------------------------------------------------
> 
> 
> http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/facb281b/log4j-core/src/main/java/org/apache/logging/log4j/core/config/LoggerConfig.java
> ----------------------------------------------------------------------
> diff --git 
> a/log4j-core/src/main/java/org/apache/logging/log4j/core/config/LoggerConfig.java
>  
> b/log4j-core/src/main/java/org/apache/logging/log4j/core/config/LoggerConfig.java
> index 02fe245..d5a03bc 100644
> --- 
> a/log4j-core/src/main/java/org/apache/logging/log4j/core/config/LoggerConfig.java
> +++ 
> b/log4j-core/src/main/java/org/apache/logging/log4j/core/config/LoggerConfig.java
> @@ -399,40 +399,6 @@ public class LoggerConfig extends AbstractFilterable {
>     /**
>      * Factory method to create a LoggerConfig.
>      *
> -     * @param additivity True if additive, false otherwise.
> -     * @param level The Level to be associated with the Logger.
> -     * @param loggerName The name of the Logger.
> -     * @param includeLocation whether location should be passed downstream
> -     * @param refs An array of Appender names.
> -     * @param properties Properties to pass to the Logger.
> -     * @param config The Configuration.
> -     * @param filter A Filter.
> -     * @return A new LoggerConfig.
> -     * @deprecated Use {@link #createLogger(boolean, Level, String, String, 
> AppenderRef[], Property[], Configuration, Filter)}
> -     */
> -    @Deprecated
> -    public static LoggerConfig createLogger(final String additivity,
> -            final Level level, @PluginAttribute("name") final String 
> loggerName,
> -            final String includeLocation,
> -            final AppenderRef[] refs,
> -            final Property[] properties, @PluginConfiguration final 
> Configuration config,
> -            final Filter filter) {
> -        if (loggerName == null) {
> -            LOGGER.error("Loggers cannot be configured without a name");
> -            return null;
> -        }
> -
> -        final List<AppenderRef> appenderRefs = Arrays.asList(refs);
> -        final String name = loggerName.equals(ROOT) ? Strings.EMPTY : 
> loggerName;
> -        final boolean additive = Booleans.parseBoolean(additivity, true);
> -
> -        return new LoggerConfig(name, appenderRefs, filter, level, additive, 
> properties, config,
> -                includeLocation(includeLocation));
> -    }
> -
> -    /**
> -     * Factory method to create a LoggerConfig.
> -     *
>      * @param additivity true if additive, false otherwise.
>      * @param level The Level to be associated with the Logger.
>      * @param loggerName The name of the Logger.
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to