Re: [log4j2] MapMessage.newInstance(Map)

2017-06-16 Thread Gary Gregory
My mistake, It looks like it is implemented and used to return an instance of its own type (different subclasses return a different type) Gary On Fri, Jun 16, 2017 at 10:30 AM, Mikael Ståldal wrote: > Maybe we should deprecate it? > > > On 2017-06-01 21:07, Gary Gregory wrote: > >> Hi All, >> >

Re: [log4j2] MapMessage.newInstance(Map)

2017-06-16 Thread Mikael Ståldal
Maybe we should deprecate it? On 2017-06-01 21:07, Gary Gregory wrote: Hi All, It looks like: org.apache.logging.log4j.message.MapMessage.newInstance(Map) was supposed to be a static method. Can't change it now I suppose :-( as it would break BC. I wonder if we should start tracking these T

[log4j2] MapMessage.newInstance(Map)

2017-06-01 Thread Gary Gregory
Hi All, It looks like: org.apache.logging.log4j.message.MapMessage.newInstance(Map) was supposed to be a static method. Can't change it now I suppose :-( as it would break BC. I wonder if we should start tracking these TODOs with // TODO Fix for 3.0 comments? Gary