Author: vsiveton Date: Tue Aug 19 04:41:08 2008 New Revision: 687025 URL: http://svn.apache.org/viewvc?rev=687025&view=rev Log: SCM-380: CvsStatusConsumer cannot be used for CvsJavaListCommand and CvsExeListCommand
o added license header o formated code Modified: maven/scm/trunk/maven-scm-providers/maven-scm-providers-cvs/maven-scm-provider-cvs-commons/src/main/java/org/apache/maven/scm/provider/cvslib/command/list/AbstractCvsListCommand.java maven/scm/trunk/maven-scm-providers/maven-scm-providers-cvs/maven-scm-provider-cvs-commons/src/main/java/org/apache/maven/scm/provider/cvslib/command/list/CvsListConsumer.java Modified: maven/scm/trunk/maven-scm-providers/maven-scm-providers-cvs/maven-scm-provider-cvs-commons/src/main/java/org/apache/maven/scm/provider/cvslib/command/list/AbstractCvsListCommand.java URL: http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-providers-cvs/maven-scm-provider-cvs-commons/src/main/java/org/apache/maven/scm/provider/cvslib/command/list/AbstractCvsListCommand.java?rev=687025&r1=687024&r2=687025&view=diff ============================================================================== --- maven/scm/trunk/maven-scm-providers/maven-scm-providers-cvs/maven-scm-provider-cvs-commons/src/main/java/org/apache/maven/scm/provider/cvslib/command/list/AbstractCvsListCommand.java (original) +++ maven/scm/trunk/maven-scm-providers/maven-scm-providers-cvs/maven-scm-provider-cvs-commons/src/main/java/org/apache/maven/scm/provider/cvslib/command/list/AbstractCvsListCommand.java Tue Aug 19 04:41:08 2008 @@ -36,6 +36,7 @@ /** * @author <a href="mailto:[EMAIL PROTECTED]">Kenney Westerhof</a> + * @version $Id$ */ public abstract class AbstractCvsListCommand extends AbstractListCommand Modified: maven/scm/trunk/maven-scm-providers/maven-scm-providers-cvs/maven-scm-provider-cvs-commons/src/main/java/org/apache/maven/scm/provider/cvslib/command/list/CvsListConsumer.java URL: http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-providers/maven-scm-providers-cvs/maven-scm-provider-cvs-commons/src/main/java/org/apache/maven/scm/provider/cvslib/command/list/CvsListConsumer.java?rev=687025&r1=687024&r2=687025&view=diff ============================================================================== --- maven/scm/trunk/maven-scm-providers/maven-scm-providers-cvs/maven-scm-provider-cvs-commons/src/main/java/org/apache/maven/scm/provider/cvslib/command/list/CvsListConsumer.java (original) +++ maven/scm/trunk/maven-scm-providers/maven-scm-providers-cvs/maven-scm-provider-cvs-commons/src/main/java/org/apache/maven/scm/provider/cvslib/command/list/CvsListConsumer.java Tue Aug 19 04:41:08 2008 @@ -1,5 +1,24 @@ package org.apache.maven.scm.provider.cvslib.command.list; +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * with the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, + * software distributed under the License is distributed on an + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY + * KIND, either express or implied. See the License for the + * specific language governing permissions and limitations + * under the License. + */ + import java.util.LinkedList; import java.util.List; @@ -11,7 +30,7 @@ /** * Parses CVS/Entries format, for example, like - * + * * <pre> * /checkoutlist/1.9/Wed Jan 26 19:08:06 2005/-kkv/ * /commitinfo/1.10/Tue Jan 11 01:25:34 2005/-kkv/ @@ -19,45 +38,56 @@ * D/directory1//// * D/directory2//// * </pre> - * + * * @author <a href="mailto:[EMAIL PROTECTED]">Sergey Zakusov</a>: implemented to fix "Unknown file status" problem + * @version $Id$ */ -public class CvsListConsumer implements StreamConsumer { - +public class CvsListConsumer + implements StreamConsumer +{ private ScmLogger m_logger; - private List m_entries; + + private List m_entries; /** * @param logger is a logger */ - public CvsListConsumer(ScmLogger logger) { - + public CvsListConsumer( ScmLogger logger ) + { m_logger = logger; m_entries = new LinkedList(); } - /** - * @see org.codehaus.plexus.util.cli.StreamConsumer#consumeLine(java.lang.String) - */ - public void consumeLine(String i_line) { - - m_logger.debug(i_line); + /** [EMAIL PROTECTED] */ + public void consumeLine( String i_line ) + { + if ( m_logger.isDebugEnabled() ) + { + m_logger.debug( i_line ); + } - String[] params = i_line.split("/"); - if (params.length < 2) { - if (StringUtils.isNotEmpty(i_line)) { - m_logger.warn("Unable to parse it as CVS/Entries format: " + i_line + "."); + String[] params = i_line.split( "/" ); + if ( params.length < 2 ) + { + if ( StringUtils.isNotEmpty( i_line ) ) + { + if ( m_logger.isWarnEnabled() ) + { + m_logger.warn( "Unable to parse it as CVS/Entries format: " + i_line + "." ); + } } - } else { - m_entries.add(new ScmFile(params[1], ScmFileStatus.UNKNOWN)); + } + else + { + m_entries.add( new ScmFile( params[1], ScmFileStatus.UNKNOWN ) ); } } /** * @return Parse result */ - public List getEntries() { - + public List getEntries() + { return m_entries; } }