Code houskeeping

Tagged some string constants as NON-NLS
Changed visibility of some logger instances for inner classes
Added qualifier for access of class variables
Organised imports
Removed unused loggers

git-svn-id: https://joinup.ec.europa.eu/svn/pdf-over/trunk@30 174cde9d-5d70-4d2a-aa98-46368bc2aaf7

21 files changed, 59 insertions, 155 deletions

diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/MainWindow.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/MainWindow.java
index 706064bd..ccf95cca 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/MainWindow.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/MainWindow.java
@@ -230,7 +230,7 @@ public class MainWindow {
 		fd_config.bottom = new FormAttachment(100, 0);
 		this.btn_config.setLayoutData(fd_config);
 		this.btn_config.setText("Config ...");
-		buttonMap.put(Buttons.CONFIG, btn_config);
+		this.buttonMap.put(Buttons.CONFIG, this.btn_config);
 
 		this.btn_open = new Button(composite, SWT.NONE);
 		FormData fd_open = new FormData();
@@ -241,7 +241,7 @@ public class MainWindow {
 		this.btn_open.setLayoutData(fd_open);
 		this.btn_open.setText("Open ...");
 		this.btn_open.addSelectionListener(new DataSourceSelectionListener());
-		buttonMap.put(Buttons.OPEN, btn_open);
+		this.buttonMap.put(Buttons.OPEN, this.btn_open);
 
 		this.btn_position = new Button(composite, SWT.NONE);
 		FormData fd_position = new FormData();
@@ -252,7 +252,7 @@ public class MainWindow {
 		this.btn_position.setLayoutData(fd_position);
 		this.btn_position.setText("Positon ...");
 		this.btn_position.addSelectionListener(new PositionSelectionListener());
-		buttonMap.put(Buttons.POSITION, btn_position);
+		this.buttonMap.put(Buttons.POSITION, this.btn_position);
 
 		this.btn_sign = new Button(composite, SWT.NONE);
 		FormData fd_sign = new FormData();
@@ -262,7 +262,7 @@ public class MainWindow {
 		fd_sign.bottom = new FormAttachment(100, 0);
 		this.btn_sign.setLayoutData(fd_sign);
 		this.btn_sign.setText("Sign ...");
-		buttonMap.put(Buttons.SIGN, btn_sign);
+		this.buttonMap.put(Buttons.SIGN, this.btn_sign);
 
 		this.container = new Composite(getShell(), SWT.BORDER | SWT.RESIZE);
 		FormData fd_composite_1 = new FormData();
@@ -299,7 +299,7 @@ public class MainWindow {
 			boolean active = behavior.getActive(button);
 			boolean enabled = behavior.getEnabled(button);
 
-			Button theButton = buttonMap.get(button);
+			Button theButton = this.buttonMap.get(button);
 			if (theButton != null)
 			{
 				theButton.setEnabled(enabled);
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/DataSourceSelectComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/DataSourceSelectComposite.java
index 44b2574b..b68ffa03 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/DataSourceSelectComposite.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/DataSourceSelectComposite.java
@@ -18,13 +18,15 @@ package at.asit.pdfover.gui.composites;
 // Imports
 import java.io.File;
 
-import org.eclipse.swt.widgets.Button;
-import org.eclipse.swt.widgets.Composite;
-import org.eclipse.swt.widgets.Display;
-import org.eclipse.swt.widgets.FileDialog;
-import org.eclipse.swt.widgets.Label;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import org.eclipse.swt.SWT;
+import org.eclipse.swt.dnd.DND;
+import org.eclipse.swt.dnd.DropTarget;
+import org.eclipse.swt.dnd.DropTargetEvent;
+import org.eclipse.swt.dnd.DropTargetListener;
+import org.eclipse.swt.dnd.FileTransfer;
+import org.eclipse.swt.dnd.Transfer;
+import org.eclipse.swt.events.SelectionEvent;
+import org.eclipse.swt.events.SelectionListener;
 import org.eclipse.swt.graphics.Color;
 import org.eclipse.swt.graphics.Font;
 import org.eclipse.swt.graphics.FontData;
@@ -32,12 +34,14 @@ import org.eclipse.swt.graphics.Point;
 import org.eclipse.swt.layout.FormAttachment;
 import org.eclipse.swt.layout.FormData;
 import org.eclipse.swt.layout.FormLayout;
-import org.eclipse.swt.SWT;
-import org.eclipse.swt.dnd.*;
-import org.eclipse.swt.events.SelectionEvent;
-import org.eclipse.swt.events.SelectionListener;
+import org.eclipse.swt.widgets.Button;
+import org.eclipse.swt.widgets.Composite;
+import org.eclipse.swt.widgets.Display;
+import org.eclipse.swt.widgets.FileDialog;
+import org.eclipse.swt.widgets.Label;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
-import at.asit.pdfover.gui.workflow.StateMachineImpl;
 import at.asit.pdfover.gui.workflow.states.State;
 
 /**
@@ -60,7 +64,7 @@ public class DataSourceSelectComposite extends StateComposite {
 		@Override
 		public void widgetSelected(SelectionEvent e) {
 			FileDialog dialog = new FileDialog(DataSourceSelectComposite.this.getShell(), SWT.OPEN);
-			dialog.setFilterExtensions(new String[] {"*.pdf"});
+			dialog.setFilterExtensions(new String[] {"*.pdf"}); //$NON-NLS-1$
 			dialog.setFilterNames(new String[] {"PDF Dateien"});
 			String fileName = dialog.open();
 	        File file = null;
@@ -81,7 +85,7 @@ public class DataSourceSelectComposite extends StateComposite {
 	/**
 	 * SFL4J Logger instance
 	 **/
-	private static final Logger log = LoggerFactory
+	static final Logger log = LoggerFactory
 			.getLogger(DataSourceSelectComposite.class);
 
 	/**
@@ -155,7 +159,7 @@ public class DataSourceSelectComposite extends StateComposite {
 						File file = new File(files[0]);
 						if(!file.exists())
 						{
-							log.error("File: " + files[0] + " doesnot exists!");
+							log.error("File: " + files[0] + " does not exist!");
 							return;
 						}
 						DataSourceSelectComposite.this.setSelected(file);
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ErrorComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ErrorComposite.java
index 6963c6f6..38b0f877 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ErrorComposite.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/ErrorComposite.java
@@ -25,8 +25,7 @@ import org.eclipse.swt.layout.FormLayout;
 import org.eclipse.swt.widgets.Button;
 import org.eclipse.swt.widgets.Composite;
 import org.eclipse.swt.widgets.Label;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+
 import at.asit.pdfover.gui.workflow.states.State;
 
 /**
@@ -37,6 +36,12 @@ public class ErrorComposite extends StateComposite {
 	 * 
 	 */
 	private final class OkSelectionListener implements SelectionListener {
+		/**
+		 * 
+		 */
+		public OkSelectionListener() {
+		}
+
 		@Override
 		public void widgetSelected(SelectionEvent e) {
 			ErrorComposite.this.userOk = true;
@@ -49,20 +54,14 @@ public class ErrorComposite extends StateComposite {
 		}
 	}
 
-	/**
-	 * SLF4J Logger instance
-	 **/
-	private static final Logger log = LoggerFactory
-			.getLogger(ErrorComposite.class);
-
 	boolean userOk = false;
 	
 	/**
 	 * Checks if the user has clicked OK
-	 * @return
+	 * @return whether the user has clicked OK
 	 */
 	public boolean isUserOk() {
-		return userOk;
+		return this.userOk;
 	}
 
 	private Exception exception;
@@ -80,6 +79,7 @@ public class ErrorComposite extends StateComposite {
 	 * Create the composite.
 	 * @param parent
 	 * @param style
+	 * @param state 
 	 */
 	public ErrorComposite(Composite parent, int style, State state) {
 		super(parent, style, state);
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/OutputComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/OutputComposite.java
index 860cd095..6c79ed01 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/OutputComposite.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/OutputComposite.java
@@ -17,7 +17,6 @@ package at.asit.pdfover.gui.composites;
 
 // Imports
 import java.awt.Desktop;
-import org.eclipse.swt.widgets.FileDialog;
 import java.io.File;
 import java.io.FileOutputStream;
 
@@ -29,6 +28,7 @@ import org.eclipse.swt.layout.FormData;
 import org.eclipse.swt.layout.FormLayout;
 import org.eclipse.swt.widgets.Button;
 import org.eclipse.swt.widgets.Composite;
+import org.eclipse.swt.widgets.FileDialog;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -39,9 +39,15 @@ import at.asit.pdfover.signator.DocumentSource;
  * GUI component for Output State
  */
 public class OutputComposite extends StateComposite {
-	
+
 	/**
-	 * Selection Listner for save button
+	 * SLF4J Logger instance
+	 **/
+	static final Logger log = LoggerFactory
+			.getLogger(OutputComposite.class);
+
+	/**
+	 * SelectionListener for save button
 	 */
 	private final class SaveSelectionListener implements SelectionListener {
 		
@@ -56,7 +62,7 @@ public class OutputComposite extends StateComposite {
 		public void widgetSelected(SelectionEvent e) {
 			try {
 				FileDialog save = new FileDialog(OutputComposite.this.getShell(), SWT.SAVE | SWT.NATIVE);
-				save.setFilterExtensions(new String[] {"*.pdf"});
+				save.setFilterExtensions(new String[] {"*.pdf"}); //$NON-NLS-1$
 				save.setFilterNames(new String[] {"PDF Dateien"});
 				
 				String target = save.open();
@@ -132,12 +138,6 @@ public class OutputComposite extends StateComposite {
 		}
 	}
 
-	/**
-	 * SLF4J Logger instance
-	 **/
-	private static final Logger log = LoggerFactory
-			.getLogger(OutputComposite.class);
-
 	File savedFile = null;
 
 	private DocumentSource signedDocument;
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/PositioningComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/PositioningComposite.java
index d641af62..e101d281 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/PositioningComposite.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/PositioningComposite.java
@@ -22,8 +22,6 @@ import org.eclipse.swt.events.SelectionListener;
 import org.eclipse.swt.widgets.Button;
 import org.eclipse.swt.widgets.Composite;
 import org.eclipse.swt.widgets.Label;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import at.asit.pdfover.gui.workflow.states.State;
 import at.asit.pdfover.signator.SignaturePosition;
@@ -59,17 +57,11 @@ public class PositioningComposite extends StateComposite {
 		}
 	}
 
-	/**
-	 * SFL4J Logger instance
-	 **/
-	private static final Logger log = LoggerFactory
-			.getLogger(PositioningComposite.class);
-	
 	private SignaturePosition position = null;
 	
 	/**
-	 * Gets the Position
-	 * @return
+	 * Gets the position of the signature
+	 * @return the SignaturePosition
 	 */
 	public SignaturePosition getPosition() {
 		return this.position;
@@ -87,6 +79,7 @@ public class PositioningComposite extends StateComposite {
 	 * Create the composite.
 	 * @param parent
 	 * @param style
+	 * @param state 
 	 */
 	public PositioningComposite(Composite parent, int style, State state) {
 		super(parent, style, state);
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/StateComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/StateComposite.java
index 0b21e3aa..dcc82394 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/StateComposite.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/StateComposite.java
@@ -17,8 +17,6 @@ package at.asit.pdfover.gui.composites;
 
 // Imports
 import org.eclipse.swt.widgets.Composite;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 import at.asit.pdfover.gui.workflow.states.State;
 
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/WaitingComposite.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/WaitingComposite.java
index 4101e924..1b609dc9 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/WaitingComposite.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/composites/WaitingComposite.java
@@ -16,16 +16,13 @@
 package at.asit.pdfover.gui.composites;
 
 // Imports
-import org.eclipse.swt.widgets.Composite;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-import org.eclipse.swt.widgets.ProgressBar;
 import org.eclipse.swt.SWT;
-import org.eclipse.swt.layout.FormLayout;
-import org.eclipse.swt.layout.FormData;
 import org.eclipse.swt.layout.FormAttachment;
+import org.eclipse.swt.layout.FormData;
+import org.eclipse.swt.layout.FormLayout;
+import org.eclipse.swt.widgets.Composite;
+import org.eclipse.swt.widgets.ProgressBar;
 
-import at.asit.pdfover.gui.workflow.StateMachineImpl;
 import at.asit.pdfover.gui.workflow.states.State;
 
 /**
@@ -33,12 +30,6 @@ import at.asit.pdfover.gui.workflow.states.State;
  */
 public class WaitingComposite extends StateComposite {
 	/**
-	 * SLF4J Logger instance
-	 **/
-	private static final Logger log = LoggerFactory
-			.getLogger(WaitingComposite.class);
-
-	/**
 	 * Create the composite.
 	 * @param parent
 	 * @param style
diff --git a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/LocalBKUState.java b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/LocalBKUState.java
index ccb411be..c1586d45 100644
--- a/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/LocalBKUState.java
+++ b/pdf-over-gui/src/main/java/at/asit/pdfover/gui/workflow/states/LocalBKUState.java
@@ -16,15 +16,15 @@
 package at.asit.pdfover.gui.workflow.states;
 
 // Imports
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-import org.apache.commons.codec.binary.Base64;
 import org.apache.commons.httpclient.HttpClient;
 import org.apache.commons.httpclient.HttpException;
 import org.apache.commons.httpclient.HttpStatus;
 import org.apache.commons.httpclient.methods.PostMethod;
-import at.asit.pdfover.gui.MainWindowBehavior;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 import at.asit.pdfover.gui.MainWindow.Buttons;
+import at.asit.pdfover.gui.MainWindowBehavior;
 import at.asit.pdfover.gui.workflow.StateMachine;
 import at.asit.pdfover.gui.workflow.Status;
 import at.asit.pdfover.signator.SLRequest;
diff --git a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/ByteArrayDocumentSource.java b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/ByteArrayDocumentSource.java
index add8ca4b..8ff79374 100644
--- a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/ByteArrayDocumentSource.java
+++ b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/ByteArrayDocumentSource.java
@@ -16,9 +16,6 @@
 package at.asit.pdfover.signator;
 
 //Imports
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import java.io.ByteArrayInputStream;
 import java.io.InputStream;
 
@@ -28,11 +25,6 @@ import java.io.InputStream;
 public class ByteArrayDocumentSource implements DocumentSource {
 
 	/**
-	 * SFL4J Logger instance
-	 **/
-	private static final Logger log = LoggerFactory.getLogger(ByteArrayDocumentSource.class);
-	
-	/**
 	 * Document content
 	 */
 	protected byte[] data;
diff --git a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/PDFFileDocumentSource.java b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/PDFFileDocumentSource.java
index 04fab03a..6fdbfe29 100644
--- a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/PDFFileDocumentSource.java
+++ b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/PDFFileDocumentSource.java
@@ -18,7 +18,6 @@ package at.asit.pdfover.signator;
 // Imports
 import java.io.File;
 import java.io.FileInputStream;
-import java.io.FileNotFoundException;
 import java.io.IOException;
 import java.io.InputStream;
 
diff --git a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignResultImpl.java b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignResultImpl.java
index a8f8b63a..3fb15bba 100644
--- a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignResultImpl.java
+++ b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignResultImpl.java
@@ -15,22 +15,14 @@
  */
 package at.asit.pdfover.signator;
 
-import java.security.cert.X509Certificate;
-
 //Imports
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import java.security.cert.X509Certificate;
 
 /**
  * The result of a signature operation
  */
 public class SignResultImpl implements SignResult {
 
-	/**
-	 * SFL4J Logger instance
-	 **/
-	private static Logger log = LoggerFactory.getLogger(Signator.class);
-	
 	private SignaturePosition position;
 	private DocumentSource source;
 	private X509Certificate certificate;
diff --git a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignatureDimension.java b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignatureDimension.java
index d6d68d44..3b5c2969 100644
--- a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignatureDimension.java
+++ b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignatureDimension.java
@@ -16,8 +16,6 @@
 package at.asit.pdfover.signator;
 
 //Imports
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 /**
  * The dimensions of the visible signature block
@@ -25,11 +23,6 @@ import org.slf4j.LoggerFactory;
 public class SignatureDimension {
 
 	/**
-	 * SFL4J Logger instance
-	 **/
-	private static Logger log = LoggerFactory.getLogger(SignatureDimension.class);
-
-	/**
 	 * The visible Signature block width
 	 */
 	protected int width;
diff --git a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignatureException.java b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignatureException.java
index 80a8ed5c..8feaee56 100644
--- a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignatureException.java
+++ b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignatureException.java
@@ -16,8 +16,6 @@
 package at.asit.pdfover.signator;
 
 //Imports
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 /**
  * Base class for signature exceptions
@@ -25,11 +23,6 @@ import org.slf4j.LoggerFactory;
 public class SignatureException extends Exception {
 
 	/**
-	 * SFL4J Logger instance
-	 **/
-	private static Logger log = LoggerFactory.getLogger(SignatureException.class);
-	
-	/**
 	 * 
 	 */
 	private static final long serialVersionUID = 711578398780816710L;
diff --git a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignatureParameter.java b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignatureParameter.java
index 1edb997b..f3ba2a34 100644
--- a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignatureParameter.java
+++ b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignatureParameter.java
@@ -16,8 +16,6 @@
 package at.asit.pdfover.signator;
 
 //Imports
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 /**
  * The Signature Parameter
@@ -25,11 +23,6 @@ import org.slf4j.LoggerFactory;
 public abstract class SignatureParameter {
 
 	/**
-	 * SFL4J Logger instance
-	 **/
-	private static Logger log = LoggerFactory.getLogger(SignatureParameter.class);
-	
-	/**
 	 * The Signature Position
 	 * @uml.property  name="signaturePosition"
 	 * @uml.associationEnd  
diff --git a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignaturePosition.java b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignaturePosition.java
index 16aae296..edcb263d 100644
--- a/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignaturePosition.java
+++ b/pdf-over-signator/src/main/java/at/asit/pdfover/signator/SignaturePosition.java
@@ -16,19 +16,12 @@
 package at.asit.pdfover.signator;
 
 //Imports
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 /**
  * Represents the position of a visible signature block
  */
 public class SignaturePosition {
-	
-	/**
-	 * SFL4J Logger instance
-	 **/
-	private static Logger log = LoggerFactory.getLogger(SignaturePosition.class);
-	
+
 	/**
 	 * The x value of the position
 	 */
diff --git a/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/ByteArrayPDFASDataSource.java b/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/ByteArrayPDFASDataSource.java
index 31e522db..5998055a 100644
--- a/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/ByteArrayPDFASDataSource.java
+++ b/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/ByteArrayPDFASDataSource.java
@@ -19,9 +19,6 @@ package at.asit.pdfover.signer.pdfas;
 import java.io.ByteArrayInputStream;
 import java.io.InputStream;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import at.gv.egiz.pdfas.api.io.DataSource;
 
 /**
@@ -30,11 +27,6 @@ import at.gv.egiz.pdfas.api.io.DataSource;
 public class ByteArrayPDFASDataSource implements DataSource {
 
 	/**
-	 * SFL4J Logger instance
-	 **/
-	private static final Logger log = LoggerFactory.getLogger(ByteArrayPDFASDataSource.class);
-	
-	/**
 	 * Internal data byte array
 	 */
 	private byte[] data;
diff --git a/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASHelper.java b/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASHelper.java
index 80617989..cac338ba 100644
--- a/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASHelper.java
+++ b/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASHelper.java
@@ -18,8 +18,6 @@ package at.asit.pdfover.signer.pdfas;
 //Imports
 import java.io.File;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import at.asit.pdfover.signator.SignatureException;
 import at.gv.egiz.pdfas.api.PdfAs;
 import at.gv.egiz.pdfas.api.exceptions.PdfAsException;
@@ -29,12 +27,7 @@ import at.gv.egiz.pdfas.api.internal.PdfAsInternal;
  * Encapsulates PDF AS API Object to need just one initialization
  */
 public class PDFASHelper {
-	
-	/**
-	 * SFL4J Logger instance
-	 **/
-	private static final Logger log = LoggerFactory.getLogger(PDFASHelper.class);
-	
+
 	/**
 	 * PDF AS Object
 	 */
diff --git a/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASSLRequest.java b/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASSLRequest.java
index 7267db85..493b3e9c 100644
--- a/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASSLRequest.java
+++ b/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASSLRequest.java
@@ -20,7 +20,6 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 import at.asit.pdfover.signator.ByteArrayDocumentSource;
-import at.asit.pdfover.signator.DocumentSource;
 import at.asit.pdfover.signator.SLRequest;
 
 /**
diff --git a/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASSignerFactory.java b/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASSignerFactory.java
index 94c13412..68a29341 100644
--- a/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASSignerFactory.java
+++ b/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASSignerFactory.java
@@ -16,8 +16,6 @@
 package at.asit.pdfover.signer.pdfas;
 
 //Imports
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import at.asit.pdfover.signator.Signer;
 import at.asit.pdfover.signator.SignerFactory;
 
@@ -26,11 +24,6 @@ import at.asit.pdfover.signator.SignerFactory;
  */
 public class PDFASSignerFactory extends SignerFactory {
 
-	/**
-	 * SFL4J Logger instance
-	 **/
-	private static final Logger log = LoggerFactory.getLogger(PDFASSignerFactory.class);
-	
 	@Override
 	public Signer createSigner() {
 		return new PDFASSigner();
diff --git a/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASSigningState.java b/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASSigningState.java
index 72e24747..4849d26e 100644
--- a/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASSigningState.java
+++ b/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PDFASSigningState.java
@@ -16,8 +16,6 @@
 package at.asit.pdfover.signer.pdfas;
 
 //Imports
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import at.asit.pdfover.signator.SLRequest;
 import at.asit.pdfover.signator.SLResponse;
 import at.asit.pdfover.signator.SigningState;
@@ -30,11 +28,6 @@ import at.gv.egiz.pdfas.api.sign.SignatureDetailInformation;
 public class PDFASSigningState implements SigningState {
 
 	/**
-	 * SFL4J Logger instance
-	 **/
-	private static final Logger log = LoggerFactory.getLogger(PDFASSigningState.class);
-	
-	/**
 	 * The Signature Layer request
 	 */
 	protected SLRequest slrequest;
diff --git a/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PdfAsSignatureParameter.java b/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PdfAsSignatureParameter.java
index 4825785b..9aec98f7 100644
--- a/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PdfAsSignatureParameter.java
+++ b/pdf-over-signer/pdf-over-sigpdfas/src/main/java/at/asit/pdfover/signer/pdfas/PdfAsSignatureParameter.java
@@ -16,23 +16,16 @@
 package at.asit.pdfover.signer.pdfas;
 
 //Imports
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import at.asit.pdfover.signator.SignatureDimension;
 import at.asit.pdfover.signator.SignatureParameter;
-import at.gv.egiz.pdfas.api.sign.pos.SignaturePositioning;
 import at.gv.egiz.pdfas.api.io.DataSource;
+import at.gv.egiz.pdfas.api.sign.pos.SignaturePositioning;
 
 /**
  * Implementation of SignatureParameter specific for PDF - AS Library
  */
 public class PdfAsSignatureParameter extends SignatureParameter {
 
-	/**
-	 * SFL4J Logger instance
-	 **/
-	private static final Logger log = LoggerFactory.getLogger(PdfAsSignatureParameter.class);
-	
 	@Override
 	public SignatureDimension getPlaceholderDimension() {
 		// TODO Auto-generated method stub