protected synchronized void createListOfEntries ( Container container ) { listOfEntries = new ArrayList < String > ( ) ; for ( Entry entry : container ) { String entryName = entry . getName ( ) ; try { Container dirContainer = entry . adapt ( Container . class ) ; if ( dirContainer != null && entry . getSize ( ) == 0 ) { if ( ! "META-INF" . equalsIgnoreCase ( entry . getName ( ) ) ) { addToList ( listOfEntries , dirContainer , "/" + entryName ) ; } } else { //only get here if the object wasn't able to be wrapped as a container - not a dir //I'm not sure why there isn't a WEB-INF check either... if ( isJspFile ( entryName , extList ) && entry . getSize ( ) >= this . _minLength ) { listOfEntries . add ( "/" + entryName ) ; } } } catch ( UnableToAdaptException e ) { //if (com.ibm.ejs.ras.TraceComponent.isAnyTracingEnabled()&&logger.isLoggable(Level.FINER)) { // logger.logp(Level.FINER, CLASS_NAME, "run", entryName + " is not a directory"); //} } } if ( listOfEntries != null ) { listOfEntriesIterator = listOfEntries . iterator ( ) ; } }
only called by single thread before others are spawned but going to synchronize to fix findbugs