[ 
https://issues.apache.org/jira/browse/LOG4NET-327?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13238068#comment-13238068
 ] 

Ron Grabowski commented on LOG4NET-327:
---------------------------------------

Why do you think that? The break means the switch statement will exit and 
return control to the while loop.
                
> in AppenderSkeleton.Append should check next Filter while got a 
> FilterDecision.Neutral
> --------------------------------------------------------------------------------------
>
>                 Key: LOG4NET-327
>                 URL: https://issues.apache.org/jira/browse/LOG4NET-327
>             Project: Log4net
>          Issue Type: Bug
>          Components: Appenders
>    Affects Versions: 1.2.9
>            Reporter: LoyeMei
>              Labels: AppenderSkeleton.Append
>
> public void DoAppend(LoggingEvent loggingEvent) 
> {
>        ....................................
>       while(f != null) 
>       {
>               switch(f.Decide(loggingEvent)) 
>               {
>                       case FilterDecision.Deny: 
>                               return;         // Return without appending
>                       case FilterDecision.Accept:
>                               f = null;       // Break out of the loop
>                               break;
>                       case FilterDecision.Neutral:
>                               f = f.Next;     // Move to next filter
>                               break;                    //Error here ,It 
> should be "continue" here
>               }
>       }
>       ...............         
> }

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to