[Jcommander-commit] plugins/org.jcommander.phileas.bookmarks/src/org/jcommander/phileas/bookmarks BookmarksFileProvider.java,1.4,1.5 BookmarksFileSystem.java,1.4,1.5

[Robert E. <sky...@us...>]

Update of /cvsroot/jcommander/plugins/org.jcommander.phileas.bookmarks/src/org/jcommander/phileas/bookmarks
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv25118/src/org/jcommander/phileas/bookmarks

Modified Files:
	BookmarksFileProvider.java BookmarksFileSystem.java 
Log Message:
Replaced System.out.println and printStackTrace() calls with Log4j logging statements so that every error gets into the log.

Index: BookmarksFileSystem.java
===================================================================
RCS file: /cvsroot/jcommander/plugins/org.jcommander.phileas.bookmarks/src/org/jcommander/phileas/bookmarks/BookmarksFileSystem.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -C2 -d -r1.4 -r1.5
*** BookmarksFileSystem.java	15 Feb 2006 09:04:39 -0000	1.4
--- BookmarksFileSystem.java	25 Apr 2006 09:27:52 -0000	1.5
***************
*** 11,14 ****
--- 11,15 ----
  import org.apache.commons.vfs.*;
  import org.apache.commons.vfs.provider.*;
+ import org.apache.log4j.Logger;
  import org.jcommander.ui.logger.*;
  import org.jcommander.vfsextensions.*;
***************
*** 18,21 ****
--- 19,24 ----
  public class BookmarksFileSystem extends AbstractFileSystem {
  
+ 	protected Logger logger = Logger.getLogger(getClass());
+ 	
  	public static final String JCMD_BMK_SCHEME = "jcmdbmk";
  	
***************
*** 88,99 ****
  
  		} catch (TransformerConfigurationException e) {
! 			// TODO Auto-generated catch block
! 			e.printStackTrace();
  		} catch (FileNotFoundException e) {
! 			// TODO Auto-generated catch block
! 			e.printStackTrace();
  		} catch (TransformerException e) {
! 			// TODO Auto-generated catch block
! 			e.printStackTrace();
  		}
  	}
--- 91,99 ----
  
  		} catch (TransformerConfigurationException e) {
! 			logger.error(e,e);
  		} catch (FileNotFoundException e) {
! 			logger.error(e,e);
  		} catch (TransformerException e) {
! 			logger.error(e,e);
  		}
  	}

Index: BookmarksFileProvider.java
===================================================================
RCS file: /cvsroot/jcommander/plugins/org.jcommander.phileas.bookmarks/src/org/jcommander/phileas/bookmarks/BookmarksFileProvider.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -C2 -d -r1.4 -r1.5
*** BookmarksFileProvider.java	21 Nov 2005 20:41:01 -0000	1.4
--- BookmarksFileProvider.java	25 Apr 2006 09:27:52 -0000	1.5
***************
*** 5,12 ****
--- 5,15 ----
  import org.apache.commons.vfs.*;
  import org.apache.commons.vfs.provider.*;
+ import org.apache.log4j.Logger;
  import org.jcommander.vfsextensions.*;
  
  public class BookmarksFileProvider extends AbstractOriginatingFileProvider {
  	
+ 	protected Logger logger = Logger.getLogger(getClass());
+ 	
  	protected final static Collection capabilities = Collections.unmodifiableCollection(Arrays.asList(new Capability[]
                                                       {
***************
*** 33,37 ****
  				return newFile;
  			} catch (Exception e) {
! 				e.printStackTrace();
  				return null;
  			}
--- 36,40 ----
  				return newFile;
  			} catch (Exception e) {
! 				logger.error(e,e);
  				return null;
  			}