You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@karaf.apache.org by cs...@apache.org on 2012/09/21 16:59:22 UTC

svn commit: r1388540 - in /karaf/trunk/features/core/src/main: java/org/apache/karaf/features/internal/ java/org/apache/karaf/features/internal/model/ java/org/apache/karaf/features/management/internal/ resources/OSGI-INF/blueprint/

Author: cschneider
Date: Fri Sep 21 14:59:21 2012
New Revision: 1388540

URL: http://svn.apache.org/viewvc?rev=1388540&view=rev
Log:
KARAF-608 Some cleanup and fixes to warnings

Modified:
    karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/EventAdminListener.java
    karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/RepositoryImpl.java
    karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Conditional.java
    karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Content.java
    karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Dependency.java
    karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Feature.java
    karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Features.java
    karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/package-info.java
    karaf/trunk/features/core/src/main/java/org/apache/karaf/features/management/internal/FeaturesService.java
    karaf/trunk/features/core/src/main/java/org/apache/karaf/features/management/internal/StandardEmitterMBean.java
    karaf/trunk/features/core/src/main/resources/OSGI-INF/blueprint/blueprint.xml

Modified: karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/EventAdminListener.java
URL: http://svn.apache.org/viewvc/karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/EventAdminListener.java?rev=1388540&r1=1388539&r2=1388540&view=diff
==============================================================================
--- karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/EventAdminListener.java (original)
+++ karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/EventAdminListener.java Fri Sep 21 14:59:21 2012
@@ -33,15 +33,15 @@ import org.osgi.util.tracker.ServiceTrac
  */
 public class EventAdminListener implements FeaturesListener {
 
-    private final ServiceTracker tracker;
+    private final ServiceTracker<EventAdmin, EventAdmin> tracker;
 
     public EventAdminListener(BundleContext context) {
-        tracker = new ServiceTracker(context, EventAdmin.class.getName(), null);
+        tracker = new ServiceTracker<EventAdmin, EventAdmin>(context, EventAdmin.class.getName(), null);
         tracker.open();
     }
 
     public void featureEvent(FeatureEvent event) {
-        EventAdmin eventAdmin = (EventAdmin) tracker.getService();
+        EventAdmin eventAdmin = tracker.getService();
         if (eventAdmin == null) {
             return;
         }
@@ -66,7 +66,7 @@ public class EventAdminListener implemen
     }
 
     public void repositoryEvent(RepositoryEvent event) {
-        EventAdmin eventAdmin = (EventAdmin) tracker.getService();
+        EventAdmin eventAdmin = tracker.getService();
         if (eventAdmin == null) {
             return;
         }

Modified: karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/RepositoryImpl.java
URL: http://svn.apache.org/viewvc/karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/RepositoryImpl.java?rev=1388540&r1=1388539&r2=1388540&view=diff
==============================================================================
--- karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/RepositoryImpl.java (original)
+++ karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/RepositoryImpl.java Fri Sep 21 14:59:21 2012
@@ -19,22 +19,16 @@ package org.apache.karaf.features.intern
 import java.io.IOException;
 import java.io.InputStream;
 import java.net.URI;
-import java.util.List;
 import org.apache.karaf.features.Repository;
 import org.apache.karaf.features.internal.model.Features;
 import org.apache.karaf.features.internal.model.JaxbUtil;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 /**
  * The repository implementation.
  */
 public class RepositoryImpl implements Repository {
 
-    private static final Logger LOGGER = LoggerFactory.getLogger(RepositoryImpl.class);
-    private int unnamedRepoId = 0;
     private URI uri;
-    private List<URI> repositories;
     private boolean valid;
     private Features features;
 

Modified: karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Conditional.java
URL: http://svn.apache.org/viewvc/karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Conditional.java?rev=1388540&r1=1388539&r2=1388540&view=diff
==============================================================================
--- karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Conditional.java (original)
+++ karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Conditional.java Fri Sep 21 14:59:21 2012
@@ -5,7 +5,6 @@ import java.util.Iterator;
 import java.util.List;
 import javax.xml.bind.annotation.XmlAccessType;
 import javax.xml.bind.annotation.XmlAccessorType;
-import javax.xml.bind.annotation.XmlAttribute;
 import javax.xml.bind.annotation.XmlElement;
 import javax.xml.bind.annotation.XmlType;
 import org.apache.karaf.features.Feature;

Modified: karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Content.java
URL: http://svn.apache.org/viewvc/karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Content.java?rev=1388540&r1=1388539&r2=1388540&view=diff
==============================================================================
--- karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Content.java (original)
+++ karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Content.java Fri Sep 21 14:59:21 2012
@@ -164,7 +164,8 @@ public class Content {
         return Collections.<ConfigFileInfo>unmodifiableList(getConfigfile());
     }
 
-    protected void interpolation(Properties properties) {
+    @SuppressWarnings("rawtypes")
+	protected void interpolation(Properties properties) {
         for (Enumeration e = properties.propertyNames(); e.hasMoreElements(); ) {
             String key = (String) e.nextElement();
             String val = properties.getProperty(key);

Modified: karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Dependency.java
URL: http://svn.apache.org/viewvc/karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Dependency.java?rev=1388540&r1=1388539&r2=1388540&view=diff
==============================================================================
--- karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Dependency.java (original)
+++ karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Dependency.java Fri Sep 21 14:59:21 2012
@@ -20,15 +20,11 @@
 
 package org.apache.karaf.features.internal.model;
 
-import java.util.List;
-import java.util.Map;
-
 import javax.xml.bind.annotation.XmlAccessType;
 import javax.xml.bind.annotation.XmlAccessorType;
 import javax.xml.bind.annotation.XmlAttribute;
 import javax.xml.bind.annotation.XmlType;
 import javax.xml.bind.annotation.XmlValue;
-import org.apache.karaf.features.*;
 
 
 /**

Modified: karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Feature.java
URL: http://svn.apache.org/viewvc/karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Feature.java?rev=1388540&r1=1388539&r2=1388540&view=diff
==============================================================================
--- karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Feature.java (original)
+++ karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Feature.java Fri Sep 21 14:59:21 2012
@@ -334,7 +334,8 @@ public class Feature extends Content imp
         return result;
     }
 
-    protected void interpolation(Properties properties) {
+    @SuppressWarnings("rawtypes")
+	protected void interpolation(Properties properties) {
         for (Enumeration e = properties.propertyNames(); e.hasMoreElements();) {
             String key = (String) e.nextElement();
             String val = properties.getProperty(key);

Modified: karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Features.java
URL: http://svn.apache.org/viewvc/karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Features.java?rev=1388540&r1=1388539&r2=1388540&view=diff
==============================================================================
--- karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Features.java (original)
+++ karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/Features.java Fri Sep 21 14:59:21 2012
@@ -19,7 +19,6 @@
 
 package org.apache.karaf.features.internal.model;
 
-import java.net.URI;
 import java.util.ArrayList;
 import java.util.List;
 import javax.xml.bind.annotation.XmlAccessType;
@@ -28,7 +27,6 @@ import javax.xml.bind.annotation.XmlAttr
 import javax.xml.bind.annotation.XmlRootElement;
 import javax.xml.bind.annotation.XmlSchemaType;
 import javax.xml.bind.annotation.XmlType;
-import org.apache.karaf.features.Repository;
 
 
 /**

Modified: karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/package-info.java
URL: http://svn.apache.org/viewvc/karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/package-info.java?rev=1388540&r1=1388539&r2=1388540&view=diff
==============================================================================
--- karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/package-info.java (original)
+++ karaf/trunk/features/core/src/main/java/org/apache/karaf/features/internal/model/package-info.java Fri Sep 21 14:59:21 2012
@@ -19,5 +19,3 @@
 
 @javax.xml.bind.annotation.XmlSchema(namespace = "http://karaf.apache.org/xmlns/features/v1.2.0", elementFormDefault = javax.xml.bind.annotation.XmlNsForm.QUALIFIED)
 package org.apache.karaf.features.internal.model;
-
-import org.apache.karaf.features.FeaturesNamespaces;
\ No newline at end of file

Modified: karaf/trunk/features/core/src/main/java/org/apache/karaf/features/management/internal/FeaturesService.java
URL: http://svn.apache.org/viewvc/karaf/trunk/features/core/src/main/java/org/apache/karaf/features/management/internal/FeaturesService.java?rev=1388540&r1=1388539&r2=1388540&view=diff
==============================================================================
--- karaf/trunk/features/core/src/main/java/org/apache/karaf/features/management/internal/FeaturesService.java (original)
+++ karaf/trunk/features/core/src/main/java/org/apache/karaf/features/management/internal/FeaturesService.java Fri Sep 21 14:59:21 2012
@@ -47,7 +47,7 @@ import org.osgi.framework.ServiceRegistr
 public class FeaturesService extends StandardEmitterMBean implements
     MBeanRegistration, FeaturesServiceMBean {
 
-    private ServiceRegistration registration;
+    private ServiceRegistration<FeaturesListener> registration;
 
     private BundleContext bundleContext;
 
@@ -55,8 +55,6 @@ public class FeaturesService extends Sta
 
 	private volatile long sequenceNumber = 0;
 
-	private MBeanServer server;
-
     private org.apache.karaf.features.FeaturesService featuresService;
 
     public FeaturesService() throws NotCompliantMBeanException {
@@ -65,13 +63,12 @@ public class FeaturesService extends Sta
 
     public ObjectName preRegister(MBeanServer server, ObjectName name) throws Exception {
         objectName = name;
-        this.server = server;
         return name;
     }
 
     public void postRegister(Boolean registrationDone) {
-        registration = bundleContext.registerService(FeaturesListener.class.getName(),
-            getFeaturesListener(), new Hashtable());
+        registration = bundleContext.registerService(FeaturesListener.class, 
+        		getFeaturesListener(), new Hashtable<String, String>());
     }
 
     public void preDeregister() throws Exception {

Modified: karaf/trunk/features/core/src/main/java/org/apache/karaf/features/management/internal/StandardEmitterMBean.java
URL: http://svn.apache.org/viewvc/karaf/trunk/features/core/src/main/java/org/apache/karaf/features/management/internal/StandardEmitterMBean.java?rev=1388540&r1=1388539&r2=1388540&view=diff
==============================================================================
--- karaf/trunk/features/core/src/main/java/org/apache/karaf/features/management/internal/StandardEmitterMBean.java (original)
+++ karaf/trunk/features/core/src/main/java/org/apache/karaf/features/management/internal/StandardEmitterMBean.java Fri Sep 21 14:59:21 2012
@@ -19,7 +19,8 @@ public class StandardEmitterMBean extend
 
     private final NotificationBroadcasterSupport emitter;
 
-    public StandardEmitterMBean(Class mbeanInterface) throws NotCompliantMBeanException {
+    @SuppressWarnings("rawtypes")
+	public StandardEmitterMBean(Class mbeanInterface) throws NotCompliantMBeanException {
         super(mbeanInterface);
         this.emitter = new NotificationBroadcasterSupport() {
             @Override

Modified: karaf/trunk/features/core/src/main/resources/OSGI-INF/blueprint/blueprint.xml
URL: http://svn.apache.org/viewvc/karaf/trunk/features/core/src/main/resources/OSGI-INF/blueprint/blueprint.xml?rev=1388540&r1=1388539&r2=1388540&view=diff
==============================================================================
--- karaf/trunk/features/core/src/main/resources/OSGI-INF/blueprint/blueprint.xml (original)
+++ karaf/trunk/features/core/src/main/resources/OSGI-INF/blueprint/blueprint.xml Fri Sep 21 14:59:21 2012
@@ -18,7 +18,13 @@
 
 -->
 <blueprint xmlns="http://www.osgi.org/xmlns/blueprint/v1.0.0"
-           xmlns:ext="http://aries.apache.org/blueprint/xmlns/blueprint-ext/v1.0.0">
+           xmlns:ext="http://aries.apache.org/blueprint/xmlns/blueprint-ext/v1.0.0"
+           xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+           xsi:schemaLocation="
+http://www.osgi.org/xmlns/blueprint/v1.0.0 http://www.osgi.org/xmlns/blueprint/v1.0.0/blueprint.xsd
+http://www.osgi.org/xmlns/blueprint-ext/v1.0.0 http://aries.apache.org/schemas/blueprint-ext/blueprint-ext.xsd
+"
+>
 
     <ext:property-placeholder placeholder-prefix="$(" placeholder-suffix=")"/>