You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2014/09/25 19:57:58 UTC

git commit: Minor cleanup

Repository: wicket
Updated Branches:
  refs/heads/master adcb7a632 -> 65384568a


Minor cleanup

- remove useless javadoc (only @see)
- Java 7 diamonds
- remove unused logger
- remove casting


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/65384568
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/65384568
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/65384568

Branch: refs/heads/master
Commit: 65384568a100ad9601f1ca880270ef9553d22324
Parents: adcb7a6
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Thu Sep 25 19:56:16 2014 +0200
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Thu Sep 25 19:56:16 2014 +0200

----------------------------------------------------------------------
 .../org/apache/wicket/DefaultMapperContext.java | 43 +-------------------
 ...markableListenerInterfaceRequestHandler.java |  4 --
 .../apache/wicket/pageStore/DiskDataStore.java  |  2 +-
 .../apache/wicket/settings/StoreSettings.java   |  4 +-
 4 files changed, 5 insertions(+), 48 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/65384568/wicket-core/src/main/java/org/apache/wicket/DefaultMapperContext.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/DefaultMapperContext.java b/wicket-core/src/main/java/org/apache/wicket/DefaultMapperContext.java
index ea5fce3..486dbf9 100644
--- a/wicket-core/src/main/java/org/apache/wicket/DefaultMapperContext.java
+++ b/wicket-core/src/main/java/org/apache/wicket/DefaultMapperContext.java
@@ -50,80 +50,49 @@ public class DefaultMapperContext implements IMapperContext
 	{
 		this.application = application;
 	}
-	/**
-	 * @see org.apache.wicket.core.request.mapper.IMapperContext#getBookmarkableIdentifier()
-	 */
+
 	@Override
 	public String getBookmarkableIdentifier()
 	{
 		return "bookmarkable";
 	}
 
-	/**
-	 * 
-	 * @see org.apache.wicket.core.request.mapper.IMapperContext#getNamespace()
-	 */
 	@Override
 	public String getNamespace()
 	{
 		return MarkupParser.WICKET;
 	}
 
-	/**
-	 * 
-	 * @see org.apache.wicket.core.request.mapper.IMapperContext#getPageIdentifier()
-	 */
 	@Override
 	public String getPageIdentifier()
 	{
 		return "page";
 	}
 
-	/**
-	 * 
-	 * @see org.apache.wicket.core.request.mapper.IMapperContext#getResourceIdentifier()
-	 */
 	@Override
 	public String getResourceIdentifier()
 	{
 		return "resource";
 	}
 
-	/**
-	 * 
-	 * @see org.apache.wicket.core.request.mapper.IMapperContext#getResourceReferenceRegistry()
-	 */
 	@Override
 	public ResourceReferenceRegistry getResourceReferenceRegistry()
 	{
 		return application.getResourceReferenceRegistry();
 	}
 
-	/**
-	 * 
-	 * @see org.apache.wicket.core.request.mapper.IMapperContext#requestListenerInterfaceFromString(java.lang.String)
-	 */
 	@Override
 	public RequestListenerInterface requestListenerInterfaceFromString(final String interfaceName)
 	{
 		return RequestListenerInterface.forName(interfaceName);
 	}
 
-	/**
-	 * 
-	 * @see org.apache.wicket.core.request.mapper.IMapperContext#requestListenerInterfaceToString(org.apache.wicket.RequestListenerInterface)
-	 */
 	@Override
 	public String requestListenerInterfaceToString(final RequestListenerInterface listenerInterface)
 	{
 		return listenerInterface.getName();
 	}
 
-	/**
-	 * 
-	 * @see org.apache.wicket.core.request.mapper.IPageSource#newPageInstance(java.lang.Class,
-	 *      org.apache.wicket.request.mapper.parameter.PageParameters)
-	 */
 	@Override
 	public IRequestablePage newPageInstance(final Class<? extends IRequestablePage> pageClass,
 		final PageParameters pageParameters)
@@ -138,10 +107,6 @@ public class DefaultMapperContext implements IMapperContext
 		}
 	}
 
-	/**
-	 * 
-	 * @see org.apache.wicket.core.request.mapper.IPageSource#getPageInstance(int)
-	 */
 	@Override
 	public IRequestablePage getPageInstance(final int pageId)
 	{
@@ -154,13 +119,9 @@ public class DefaultMapperContext implements IMapperContext
 		return requestablePage;
 	}
 
-	/**
-	 * 
-	 * @see org.apache.wicket.core.request.mapper.IMapperContext#getHomePageClass()
-	 */
 	@Override
 	public Class<? extends IRequestablePage> getHomePageClass()
 	{
 		return application.getHomePage();
 	}
-}
\ No newline at end of file
+}

http://git-wip-us.apache.org/repos/asf/wicket/blob/65384568/wicket-core/src/main/java/org/apache/wicket/core/request/handler/BookmarkableListenerInterfaceRequestHandler.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/core/request/handler/BookmarkableListenerInterfaceRequestHandler.java b/wicket-core/src/main/java/org/apache/wicket/core/request/handler/BookmarkableListenerInterfaceRequestHandler.java
index cc7e1b9..edc65a8 100644
--- a/wicket-core/src/main/java/org/apache/wicket/core/request/handler/BookmarkableListenerInterfaceRequestHandler.java
+++ b/wicket-core/src/main/java/org/apache/wicket/core/request/handler/BookmarkableListenerInterfaceRequestHandler.java
@@ -22,8 +22,6 @@ import org.apache.wicket.request.component.IRequestableComponent;
 import org.apache.wicket.request.component.IRequestablePage;
 import org.apache.wicket.request.mapper.parameter.PageParameters;
 import org.apache.wicket.util.lang.Args;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 /**
  * Request handler for bookmarkable pages with listener interface. This handler is only used to
@@ -36,8 +34,6 @@ public class BookmarkableListenerInterfaceRequestHandler
 		IPageRequestHandler,
 		IComponentRequestHandler
 {
-	private static final Logger logger = LoggerFactory.getLogger(BookmarkableListenerInterfaceRequestHandler.class);
-
 	private final IPageAndComponentProvider pageComponentProvider;
 
 	private final RequestListenerInterface listenerInterface;

http://git-wip-us.apache.org/repos/asf/wicket/blob/65384568/wicket-core/src/main/java/org/apache/wicket/pageStore/DiskDataStore.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/pageStore/DiskDataStore.java b/wicket-core/src/main/java/org/apache/wicket/pageStore/DiskDataStore.java
index 0858ad9..887fc09 100644
--- a/wicket-core/src/main/java/org/apache/wicket/pageStore/DiskDataStore.java
+++ b/wicket-core/src/main/java/org/apache/wicket/pageStore/DiskDataStore.java
@@ -74,7 +74,7 @@ public class DiskDataStore implements IDataStore
 		this.applicationName = applicationName;
 		this.fileStoreFolder = fileStoreFolder;
 		maxSizePerPageSession = Args.notNull(maxSizePerSession, "maxSizePerSession");
-		sessionEntryMap = new ConcurrentHashMap<String, SessionEntry>();
+		sessionEntryMap = new ConcurrentHashMap<>();
 
 		try
 		{

http://git-wip-us.apache.org/repos/asf/wicket/blob/65384568/wicket-core/src/main/java/org/apache/wicket/settings/StoreSettings.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/settings/StoreSettings.java b/wicket-core/src/main/java/org/apache/wicket/settings/StoreSettings.java
index 158a679..db52faf 100644
--- a/wicket-core/src/main/java/org/apache/wicket/settings/StoreSettings.java
+++ b/wicket-core/src/main/java/org/apache/wicket/settings/StoreSettings.java
@@ -127,9 +127,9 @@ public class StoreSettings
 	{
 		if (fileStoreFolder == null)
 		{
-			if (Application.exists())
+			if (WebApplication.exists())
 			{
-				fileStoreFolder = (File)((WebApplication)Application.get()).getServletContext()
+				fileStoreFolder = (File) WebApplication.get().getServletContext()
 					.getAttribute("javax.servlet.context.tempdir");
 			}