https://bz.apache.org/bugzilla/show_bug.cgi?id=58538

            Bug ID: 58538
           Summary: McastServiceImpl run uses bitwise and
           Product: Tomcat 8
           Version: trunk
          Hardware: Macintosh
                OS: Mac OS X 10.1
            Status: NEW
          Severity: trivial
          Priority: P2
         Component: Catalina
          Assignee: dev@tomcat.apache.org
          Reporter: anth...@whitford.com

The 'if (stopService() & startService())' looks suspicious like it should be
logical &&, not a bitwise &.


        @Override
        public void run() {
            boolean success = false;
            int attempt = 0;
            try {
                while (!success) {
                    if(log.isInfoEnabled())
                        log.info(sm.getString("mcastServiceImpl.recovery"));
                    if (stopService() & startService()) {
                        success = true;
                        if(log.isInfoEnabled())
                           
log.info(sm.getString("mcastServiceImpl.recovery.successful"));
                    }
                    try {
                        if (!success) {
                            if(log.isInfoEnabled())
                               
log.info(sm.getString("mcastServiceImpl.recovery.failed",
                                        Integer.toString(++attempt),
                                       
Long.toString(parent.recoverySleepTime)));
                            Thread.sleep(parent.recoverySleepTime);
                        }
                    }catch (InterruptedException ignore) {
                    }
                }
            }finally {
                running.set(false);
            }
        }

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to