Re: org.apache.logging.log4j.core.layout.PatternLayout.setConversionPattern(String)

2014-06-25 Thread Gary Gregory
ker wrote: > >> Sounds good. Immutability should be the default way of doing things >> anyhow. >> >> >> On 25 June 2014 10:24, Gary Gregory wrote: >> >>> Hi All: >>> >>> The method >>> org.apache.logging.log4j.core.layout.Patte

Re: org.apache.logging.log4j.core.layout.PatternLayout.setConversionPattern(String)

2014-06-25 Thread Remko Popma
2014 10:24, Gary Gregory wrote: > >> Hi All: >> >> The method >> org.apache.logging.log4j.core.layout.PatternLayout.setConversionPattern(String) >> should not exist. It is only used in one test and forces some PL state to >> be mutable. >> >> I applie

Re: org.apache.logging.log4j.core.layout.PatternLayout.setConversionPattern(String)

2014-06-25 Thread Matt Sicker
Sounds good. Immutability should be the default way of doing things anyhow. On 25 June 2014 10:24, Gary Gregory wrote: > Hi All: > > The method > org.apache.logging.log4j.core.layout.PatternLayout.setConversionPattern(String) > should not exist. It is only used in one test and

org.apache.logging.log4j.core.layout.PatternLayout.setConversionPattern(String)

2014-06-25 Thread Gary Gregory
Hi All: The method org.apache.logging.log4j.core.layout.PatternLayout.setConversionPattern(String) should not exist. It is only used in one test and forces some PL state to be mutable. I applied a fix for https://issues.apache.org/jira/browse/LOG4J2-682 that exacerbates the problem by making