[ 
https://jira.qos.ch/browse/SLF4J-432?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=19559#comment-19559
 ] 

Rufus Andrews commented on SLF4J-432:
-------------------------------------

@ceki PTAL - with respect, the delay on addressing this is getting slightly 
ridiculous. Thanks.

> log4j-over-slf4j calls wrong methods when marker parameter is null
> ------------------------------------------------------------------
>
>                 Key: SLF4J-432
>                 URL: https://jira.qos.ch/browse/SLF4J-432
>             Project: SLF4J
>          Issue Type: Bug
>          Components: log4j-over-slf4j
>            Reporter: Rufus Andrews
>            Assignee: SLF4J developers list
>            Priority: Minor
>
> The org.slf4j.Logger interface has with-Marker and without-Marker versions of 
> each logging method,
> e.g.
> public void trace(Marker marker, String format, Object arg);
> vs
> public void trace(String format, Object arg);
>  
> However, the differentiatedLog method in log4j-over-slf4j ignores this and 
> always calls the method *with* the Marker argument, even when there is no 
> Marker. Logger implementations are then forced to defensively check for null 
> Markers because of this error.
> log4j-over-slf4j should instead call the correct (without-Marker) method when 
> there is no Marker.
>  
> Patch: https://github.com/qos-ch/slf4j/pull/190/



--
This message was sent by Atlassian JIRA
(v7.3.1#73012)
_______________________________________________
slf4j-dev mailing list
slf4j-dev@qos.ch
http://mailman.qos.ch/mailman/listinfo/slf4j-dev

Reply via email to