Author: cziegeler
Date: Mon Mar 2 10:02:10 2015
New Revision: 1663237
URL: http://svn.apache.org/r1663237
Log:
FELIX-4060 : Minor refactoring; code cleanup
Added:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BuilderConstants.java
(with props)
Modified:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BaseDTOBuilder.java
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BaseServletDTOBuilder.java
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ErrorPageDTOBuilder.java
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/FilterDTOBuilder.java
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ResourceDTOBuilder.java
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/RuntimeDTOBuilder.java
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ServletContextDTOBuilder.java
Modified:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BaseDTOBuilder.java
URL:
http://svn.apache.org/viewvc/felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BaseDTOBuilder.java?rev=1663237&r1=1663236&r2=1663237&view=diff
==============================================================================
---
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BaseDTOBuilder.java
(original)
+++
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BaseDTOBuilder.java
Mon Mar 2 10:02:10 2015
@@ -22,10 +22,7 @@ import static java.util.Arrays.copyOf;
import java.util.ArrayList;
import java.util.Collection;
-import java.util.Collections;
-import java.util.HashMap;
import java.util.List;
-import java.util.Map;
import org.osgi.dto.DTO;
@@ -44,11 +41,6 @@ abstract class BaseDTOBuilder<T, U exten
abstract U buildDTO(T whiteboardService, long servletContextId);
- protected <K, V> Map<K, V> copyWithDefault(Map<K, V> map)
- {
- return map == null ? Collections.<K, V>emptyMap() : new HashMap<K,
V>(map);
- }
-
protected <V> V[] copyWithDefault(V[] array, V[] defaultArray)
{
return array == null ? defaultArray : copyOf(array, array.length);
Modified:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BaseServletDTOBuilder.java
URL:
http://svn.apache.org/viewvc/felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BaseServletDTOBuilder.java?rev=1663237&r1=1663236&r2=1663237&view=diff
==============================================================================
---
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BaseServletDTOBuilder.java
(original)
+++
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BaseServletDTOBuilder.java
Mon Mar 2 10:02:10 2015
@@ -32,7 +32,7 @@ abstract class BaseServletDTOBuilder<T,
Servlet servlet = servletHandler.getServlet();
dto.asyncSupported = info.isAsyncSupported();
- dto.initParams = copyWithDefault(info.getInitParameters());
+ dto.initParams = info.getInitParameters();
dto.name = info.getName();
dto.serviceId = servletHandler.getServletInfo().getServiceId();
dto.servletContextId = servletContextId;
Added:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BuilderConstants.java
URL:
http://svn.apache.org/viewvc/felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BuilderConstants.java?rev=1663237&view=auto
==============================================================================
---
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BuilderConstants.java
(added)
+++
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BuilderConstants.java
Mon Mar 2 10:02:10 2015
@@ -0,0 +1,40 @@
+/*
+ * 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.
+ */
+package org.apache.felix.http.base.internal.runtime.dto;
+
+import org.osgi.service.http.runtime.dto.ErrorPageDTO;
+import org.osgi.service.http.runtime.dto.FilterDTO;
+import org.osgi.service.http.runtime.dto.ListenerDTO;
+import org.osgi.service.http.runtime.dto.ResourceDTO;
+import org.osgi.service.http.runtime.dto.ServletContextDTO;
+import org.osgi.service.http.runtime.dto.ServletDTO;
+
+public abstract class BuilderConstants
+{
+
+ public static final String[] STRING_ARRAY = new String[0];
+
+ public static final ServletContextDTO[] CONTEXT_DTO_ARRAY = new
ServletContextDTO[0];
+
+ public static final ServletDTO[] SERVLET_DTO_ARRAY = new ServletDTO[0];
+ public static final ResourceDTO[] RESOURCE_DTO_ARRAY = new ResourceDTO[0];
+ public static final FilterDTO[] FILTER_DTO_ARRAY = new FilterDTO[0];
+ public static final ErrorPageDTO[] ERROR_PAGE_DTO_ARRAY = new
ErrorPageDTO[0];
+ public static final ListenerDTO[] LISTENER_DTO_ARRAY = new ListenerDTO[0];
+}
Propchange:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BuilderConstants.java
------------------------------------------------------------------------------
svn:eol-style = native
Propchange:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BuilderConstants.java
------------------------------------------------------------------------------
svn:keywords = author date id revision rev url
Propchange:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/BuilderConstants.java
------------------------------------------------------------------------------
svn:mime-type = text/plain
Modified:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ErrorPageDTOBuilder.java
URL:
http://svn.apache.org/viewvc/felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ErrorPageDTOBuilder.java?rev=1663237&r1=1663236&r2=1663237&view=diff
==============================================================================
---
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ErrorPageDTOBuilder.java
(original)
+++
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ErrorPageDTOBuilder.java
Mon Mar 2 10:02:10 2015
@@ -26,15 +26,13 @@ import org.osgi.service.http.runtime.dto
final class ErrorPageDTOBuilder extends BaseServletDTOBuilder<ErrorPage,
ErrorPageDTO>
{
- private static final String[] STRING_ARRAY = new String[0];
-
@Override
ErrorPageDTO buildDTO(ErrorPage errorPage, long servletConextId)
{
ErrorPageDTO errorPageDTO = new ErrorPageDTO();
setBaseFields(errorPageDTO, errorPage.getServletHandler(),
servletConextId);
errorPageDTO.errorCodes = getErrorCodes(errorPage.getErrorCodes());
- errorPageDTO.exceptions =
errorPage.getExceptions().toArray(STRING_ARRAY);
+ errorPageDTO.exceptions =
errorPage.getExceptions().toArray(BuilderConstants.STRING_ARRAY);
return errorPageDTO;
}
@@ -44,7 +42,7 @@ final class ErrorPageDTOBuilder extends
long[] result = new long[errorCodes.size()];
for (int i = 0; i < result.length; i++)
{
- result[i] = (long) itr.next();
+ result[i] = itr.next();
}
return result;
}
Modified:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/FilterDTOBuilder.java
URL:
http://svn.apache.org/viewvc/felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/FilterDTOBuilder.java?rev=1663237&r1=1663236&r2=1663237&view=diff
==============================================================================
---
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/FilterDTOBuilder.java
(original)
+++
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/FilterDTOBuilder.java
Mon Mar 2 10:02:10 2015
@@ -26,8 +26,6 @@ import org.osgi.service.http.runtime.dto
final class FilterDTOBuilder extends BaseDTOBuilder<FilterHandler, FilterDTO>
{
- private static final String[] STRING_ARRAY = new String[0];
-
@Override
FilterDTO buildDTO(FilterHandler filterHandler, long servletContextId)
{
@@ -36,13 +34,13 @@ final class FilterDTOBuilder extends Bas
FilterDTO filterDTO = new FilterDTO();
filterDTO.asyncSupported = info.isAsyncSupported();
filterDTO.dispatcher = getNames(info.getDispatcher());
- filterDTO.initParams = copyWithDefault(info.getInitParameters());
+ filterDTO.initParams = info.getInitParameters();
filterDTO.name = info.getName();
- filterDTO.patterns = copyWithDefault(info.getPatterns(), STRING_ARRAY);
- filterDTO.regexs = copyWithDefault(info.getRegexs(), STRING_ARRAY);
+ filterDTO.patterns = copyWithDefault(info.getPatterns(),
BuilderConstants.STRING_ARRAY);
+ filterDTO.regexs = copyWithDefault(info.getRegexs(),
BuilderConstants.STRING_ARRAY);
filterDTO.serviceId = filterHandler.getFilterInfo().getServiceId();
filterDTO.servletContextId = servletContextId;
- filterDTO.servletNames = copyWithDefault(info.getServletNames(),
STRING_ARRAY);
+ filterDTO.servletNames = copyWithDefault(info.getServletNames(),
BuilderConstants.STRING_ARRAY);
return filterDTO;
}
@@ -51,7 +49,7 @@ final class FilterDTOBuilder extends Bas
{
if (dispatcher == null)
{
- return STRING_ARRAY;
+ return BuilderConstants.STRING_ARRAY;
}
String[] names = new String[dispatcher.length];
Modified:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ResourceDTOBuilder.java
URL:
http://svn.apache.org/viewvc/felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ResourceDTOBuilder.java?rev=1663237&r1=1663236&r2=1663237&view=diff
==============================================================================
---
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ResourceDTOBuilder.java
(original)
+++
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ResourceDTOBuilder.java
Mon Mar 2 10:02:10 2015
@@ -24,15 +24,13 @@ import org.osgi.service.http.runtime.dto
final class ResourceDTOBuilder extends BaseDTOBuilder<ServletHandler,
ResourceDTO>
{
- private static final String[] STRING_ARRAY = new String[0];
-
@Override
ResourceDTO buildDTO(ServletHandler handler, long servletContextId)
{
ServletInfo servletInfo = handler.getServletInfo();
ResourceDTO resourceDTO = new ResourceDTO();
- resourceDTO.patterns = copyWithDefault(servletInfo.getPatterns(),
STRING_ARRAY);
+ resourceDTO.patterns = copyWithDefault(servletInfo.getPatterns(),
BuilderConstants.STRING_ARRAY);
resourceDTO.prefix = servletInfo.getPrefix();
resourceDTO.serviceId = servletInfo.getServiceId();
resourceDTO.servletContextId = servletContextId;
Modified:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/RuntimeDTOBuilder.java
URL:
http://svn.apache.org/viewvc/felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/RuntimeDTOBuilder.java?rev=1663237&r1=1663236&r2=1663237&view=diff
==============================================================================
---
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/RuntimeDTOBuilder.java
(original)
+++
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/RuntimeDTOBuilder.java
Mon Mar 2 10:02:10 2015
@@ -51,7 +51,6 @@ import org.osgi.service.http.runtime.dto
public final class RuntimeDTOBuilder
{
- private static final ServletContextDTO[] CONTEXT_DTO_ARRAY = new
ServletContextDTO[0];
private static final ServletDTOBuilder SERVLET_DTO_BUILDER = new
ServletDTOBuilder();
private static final ResourceDTOBuilder RESOURCE_DTO_BUILDER = new
ResourceDTOBuilder();
@@ -87,15 +86,15 @@ public final class RuntimeDTOBuilder
final FailedServletContextDTO dto = new
FailedServletContextDTO();
dto.attributes = Collections.emptyMap();
dto.contextPath = sch.getPath();
- dto.errorPageDTOs = new ErrorPageDTO[0];
+ dto.errorPageDTOs = BuilderConstants.ERROR_PAGE_DTO_ARRAY;
dto.failureReason =
DTOConstants.FAILURE_REASON_VALIDATION_FAILED;
- dto.filterDTOs = new FilterDTO[0];
+ dto.filterDTOs = BuilderConstants.FILTER_DTO_ARRAY;
dto.initParams = sch.getInitParameters();
- dto.listenerDTOs = new ListenerDTO[0];
+ dto.listenerDTOs = BuilderConstants.LISTENER_DTO_ARRAY;
dto.name = sch.getName();
- dto.resourceDTOs = new ResourceDTO[0];
+ dto.resourceDTOs = BuilderConstants.RESOURCE_DTO_ARRAY;
dto.serviceId = sch.getServiceId();
- dto.servletDTOs = new ServletDTO[0];
+ dto.servletDTOs = BuilderConstants.SERVLET_DTO_ARRAY;
failedServletContextDTOs.add(dto);
}
@@ -131,7 +130,7 @@ public final class RuntimeDTOBuilder
registry.getHandlerRuntime(context),
registry.getListenerRuntime(context)));
}
- return contextDTOs.toArray(CONTEXT_DTO_ARRAY);
+ return contextDTOs.toArray(BuilderConstants.CONTEXT_DTO_ARRAY);
}
private ServletContextDTO createContextDTO(ContextHandler context,
Modified:
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ServletContextDTOBuilder.java
URL:
http://svn.apache.org/viewvc/felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ServletContextDTOBuilder.java?rev=1663237&r1=1663236&r2=1663237&view=diff
==============================================================================
---
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ServletContextDTOBuilder.java
(original)
+++
felix/trunk/http/base/src/main/java/org/apache/felix/http/base/internal/runtime/dto/ServletContextDTOBuilder.java
Mon Mar 2 10:02:10 2015
@@ -38,11 +38,6 @@ import org.osgi.service.http.runtime.dto
final class ServletContextDTOBuilder
{
- private static final ServletDTO[] SERVLET_DTO_ARRAY = new ServletDTO[0];
- private static final ResourceDTO[] RESOURCE_DTO_ARRAY = new ResourceDTO[0];
- private static final FilterDTO[] FILTER_DTO_ARRAY = new FilterDTO[0];
- private static final ErrorPageDTO[] ERROR_PAGE_DTO_ARRAY = new
ErrorPageDTO[0];
- private static final ListenerDTO[] LISTENER_DTO_ARRAY = new ListenerDTO[0];
private final ContextHandler contextHandler;
private final ServletDTO[] servletDTOs;
@@ -60,15 +55,15 @@ final class ServletContextDTOBuilder
{
this.contextHandler = contextHandler;
this.servletDTOs = servletDTOs != null ?
- servletDTOs.toArray(SERVLET_DTO_ARRAY) : SERVLET_DTO_ARRAY;
+ servletDTOs.toArray(BuilderConstants.SERVLET_DTO_ARRAY) :
BuilderConstants.SERVLET_DTO_ARRAY;
this.resourceDTOs = resourceDTOs != null ?
- resourceDTOs.toArray(RESOURCE_DTO_ARRAY) : RESOURCE_DTO_ARRAY;
+ resourceDTOs.toArray(BuilderConstants.RESOURCE_DTO_ARRAY) :
BuilderConstants.RESOURCE_DTO_ARRAY;
this.filterDTOs = filterDTOs != null ?
- filterDTOs.toArray(FILTER_DTO_ARRAY) : FILTER_DTO_ARRAY;
+ filterDTOs.toArray(BuilderConstants.FILTER_DTO_ARRAY) :
BuilderConstants.FILTER_DTO_ARRAY;
this.errorPageDTOs = errorPageDTOs != null ?
- errorPageDTOs.toArray(ERROR_PAGE_DTO_ARRAY) :
ERROR_PAGE_DTO_ARRAY;
+ errorPageDTOs.toArray(BuilderConstants.ERROR_PAGE_DTO_ARRAY) :
BuilderConstants.ERROR_PAGE_DTO_ARRAY;
this.listenerDTOs = listenerDTOs != null ?
- listenerDTOs.toArray(LISTENER_DTO_ARRAY) : LISTENER_DTO_ARRAY;
+ listenerDTOs.toArray(BuilderConstants.LISTENER_DTO_ARRAY) :
BuilderConstants.LISTENER_DTO_ARRAY;
}
ServletContextDTO build()
@@ -82,12 +77,13 @@ final class ServletContextDTOBuilder
contextDTO.contextPath = context.getContextPath();
contextDTO.errorPageDTOs = errorPageDTOs;
contextDTO.filterDTOs = filterDTOs;
- contextDTO.initParams = getInitParameters(context);
+ contextDTO.initParams = contextInfo.getInitParameters();
contextDTO.listenerDTOs = listenerDTOs;
contextDTO.name = context.getServletContextName();
contextDTO.resourceDTOs = resourceDTOs;
contextDTO.servletDTOs = servletDTOs;
contextDTO.serviceId = contextId;
+
return contextDTO;
}
@@ -124,14 +120,4 @@ final class ServletContextDTOBuilder
byte.class.isAssignableFrom(type) ||
char.class.isAssignableFrom(type);
}
-
- private Map<String, String> getInitParameters(ServletContext context)
- {
- Map<String, String> initParameters = new HashMap<String, String>();
- for (String name : list(context.getInitParameterNames()))
- {
- initParameters.put(name, context.getInitParameter(name));
- }
- return initParameters;
- }
}
