lttng ui: Fix minor sonar warnings
[deliverable/tracecompass.git] / org.eclipse.linuxtools.lttng2.ui / src / org / eclipse / linuxtools / internal / lttng2 / ui / views / control / dialogs / EnableKernelEventComposite.java
index ee808066264ffcbda7ae989cc000834fea62c8ec..812e5cfb45b1c2ff212022d37d56382ad5a0a4bb 100644 (file)
@@ -1,12 +1,12 @@
 /**********************************************************************
- * Copyright (c) 2012 Ericsson
- * 
+ * Copyright (c) 2012, 2013 Ericsson
+ *
  * All rights reserved. This program and the accompanying materials are
  * made available under the terms of the Eclipse Public License v1.0 which
  * accompanies this distribution, and is available at
  * http://www.eclipse.org/legal/epl-v10.html
- * 
- * Contributors: 
+ *
+ * Contributors:
  *   Bernd Hufmann - Initial API and implementation
  **********************************************************************/
 package org.eclipse.linuxtools.internal.lttng2.ui.views.control.dialogs;
@@ -41,7 +41,7 @@ import org.eclipse.swt.widgets.Text;
  * <p>
  * A composite for collecting information about kernel events to be enabled.
  * </p>
- * 
+ *
  * @author Bernd Hufmann
  */
 public class EnableKernelEventComposite extends Composite implements IEnableKernelEvents {
@@ -49,7 +49,7 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
     // ------------------------------------------------------------------------
     // Constants
     // ------------------------------------------------------------------------
-    private enum KernelGroupEnum { TRACEPOINTS, SYSCALLS, PROBE, FUNCTION };
+    private enum KernelGroupEnum { TRACEPOINTS, SYSCALLS, PROBE, FUNCTION }
 
     // ------------------------------------------------------------------------
     // Attributes
@@ -72,7 +72,7 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
      */
     private Button fProbeActivateButton;
     /**
-     * The text field for the event name for the dynamic probe. 
+     * The text field for the event name for the dynamic probe.
      */
     private Text fProbeEventNameText;
     /**
@@ -84,7 +84,7 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
      */
     private Button fFunctionActivateButton;
     /**
-     * The text field for the event name for the dynamic probe. 
+     * The text field for the event name for the dynamic probe.
      */
     private Text fFunctionEventNameText;
     /**
@@ -92,10 +92,10 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
      */
     private Text fFunctionText;
     /**
-     * The referenced trace provider group containing the kernel provider 
+     * The referenced trace provider group containing the kernel provider
      * component which contains a list of available tracepoints.
      */
-    private TraceProviderGroup fProviderGroup;
+    private final TraceProviderGroup fProviderGroup;
     /**
      * The flag indicating that tracepoints are selected.
      */
@@ -117,7 +117,7 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
      */
     private boolean fIsDynamicProbe;
     /**
-     *  The event name of the dynamic probe. 
+     *  The event name of the dynamic probe.
      */
     private String fProbeEventName;
     /**
@@ -141,6 +141,16 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
     // Constructors
     // ------------------------------------------------------------------------
 
+    /**
+     * Constructor
+     *
+     * @param parent
+     *            The parent composite
+     * @param style
+     *            The index of the style for this event composite
+     * @param providerGroup
+     *            The trace provider group
+     */
     public EnableKernelEventComposite(Composite parent, int style, TraceProviderGroup providerGroup) {
         super(parent, style);
         fProviderGroup = providerGroup;
@@ -149,92 +159,57 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
     // ------------------------------------------------------------------------
     // Acessors
     // ------------------------------------------------------------------------
-    /*
-     * (non-Javadoc)
-     * @see org.eclipse.linuxtools.internal.lttng2.ui.views.control.dialogs.IEnableKernelEvents#isTracpoints()
-     */
+
     @Override
     public boolean isTracepoints() {
         return fIsTracepoints;
     }
-    
-    /* (non-Javadoc)
-     * @see org.eclipse.linuxtools.internal.lttng2.ui.views.control.dialogs.IEnableKernelEvents#isAllTracePoints()
-     */
+
     @Override
     public boolean isAllTracePoints() {
         return fIsAllTracepoints;
     }
-    
-    /*
-     * (non-Javadoc)
-     * @see org.eclipse.linuxtools.internal.lttng2.ui.views.control.dialogs.IEnableKernelEvents#isSysCalls()
-     */
+
     @Override
     public boolean isSysCalls() {
         return fIsSysCalls;
     }
-    
-    /* (non-Javadoc)
-     * @see org.eclipse.linuxtools.internal.lttng2.ui.views.control.dialogs.IEnableKernelEvents#isAllSysCalls()
-     */
+
     @Override
     public boolean isAllSysCalls() {
         return fIsSysCalls;
     }
-    
-    /* (non-Javadoc)
-     * @see org.eclipse.linuxtools.internal.lttng2.ui.views.control.dialogs.IEnableKernelEvents#getEventNames()
-     */
+
     @Override
     public List<String> getEventNames() {
         return new ArrayList<String>(fSelectedEvents);
     }
-    /*
-     * (non-Javadoc)
-     * @see org.eclipse.linuxtools.internal.lttng2.ui.views.control.dialogs.IEnableKernelEvents#isDynamicProbe()
-     */
+
     @Override
     public boolean isDynamicProbe() {
         return fIsDynamicProbe;
     }
-    
-    /* (non-Javadoc)
-     * @see org.eclipse.linuxtools.internal.lttng2.ui.views.control.dialogs.IEnableKernelEvents#getProbeName()
-     */
+
     @Override
     public String getProbeName() {
         return fProbeString;
     }
-    
-    /* (non-Javadoc)
-     * @see org.eclipse.linuxtools.internal.lttng2.ui.views.control.dialogs.IEnableKernelEvents#getProbeEventName()
-     */
+
     @Override
     public String getProbeEventName() {
         return fProbeEventName;
     }
-    
-    /*
-     * (non-Javadoc)
-     * @see org.eclipse.linuxtools.internal.lttng2.ui.views.control.dialogs.IEnableKernelEvents#isDynamicFunctionProbe()
-     */
+
     @Override
     public boolean isDynamicFunctionProbe() {
         return fIsDynamicFunctionProbe;
     }
-    
-    /* (non-Javadoc)
-     * @see org.eclipse.linuxtools.internal.lttng2.ui.views.control.dialogs.IEnableKernelEvents#getFunctionEventName()
-     */
+
     @Override
     public String getFunctionEventName() {
         return fFunctionEventName;
     }
-    
-    /* (non-Javadoc)
-     * @see org.eclipse.linuxtools.internal.lttng2.ui.views.control.dialogs.IEnableKernelEvents#getFunction()
-     */
+
     @Override
     public String getFunction() {
         return fFunctionString;
@@ -243,27 +218,28 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
     // ------------------------------------------------------------------------
     // Operations
     // ------------------------------------------------------------------------
+
     /**
      * Creates the composite content
      */
     public void createContent() {
-        
-        // Tracepoints Group 
+
+        // Tracepoints Group
         createTracepointsGroup();
 
-        // Syscalls Group 
+        // Syscalls Group
         createSysCallsGroup();
-        
-        // Dynamic Probe Group 
+
+        // Dynamic Probe Group
         createDynamicProbeGroup();
 
-        // Dynamic Function Probe Group 
+        // Dynamic Function Probe Group
         createDynamicFunctionPropeGroup();
-        
+
         // Set default enablements
         setKernelEnablements(KernelGroupEnum.TRACEPOINTS);
     }
-    
+
     /**
      * Validates the kernel composite input data.
      * @return true if configured data is valid and can be retrieved.
@@ -305,7 +281,7 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
                 fProbeEventName = temp;
                 // fProbeString will be validated by lttng-tools
                 fProbeString = fProbeText.getText();
-            } 
+            }
         }
 
         // initialize function string
@@ -330,7 +306,7 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
 
         return true;
     }
-    
+
     /**
      * Creates tracepoints group.
      */
@@ -559,14 +535,14 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
     // Local classes
     // ------------------------------------------------------------------------
     /**
-     * Content provider for the tracepoints tree. 
+     * Content provider for the tracepoints tree.
      */
-    final static public class KernelContentProvider extends TraceControlContentProvider {
+    public static final class KernelContentProvider extends TraceControlContentProvider {
         @Override
         public Object[] getChildren(Object parentElement) {
             if (parentElement instanceof TraceProviderGroup) {
                 List<ITraceControlComponent> children = ((ITraceControlComponent)parentElement).getChildren(KernelProviderComponent.class);
-                return (ITraceControlComponent[]) children.toArray(new ITraceControlComponent[children.size()]);
+                return children.toArray(new ITraceControlComponent[children.size()]);
             }
             if (parentElement instanceof ITraceControlComponent) {
                 return ((ITraceControlComponent)parentElement).getChildren();
@@ -574,11 +550,11 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
             return new Object[0];
         }
     }
-    
+
     /**
-     * Content label for the tracepoints tree. 
+     * Content label for the tracepoints tree.
      */
-    final static public class KernelLabelProvider extends TraceControlLabelProvider {
+    public static final class KernelLabelProvider extends TraceControlLabelProvider {
         @Override
         public Image getImage(Object element) {
             return null;
@@ -591,18 +567,18 @@ public class EnableKernelEventComposite extends Composite implements IEnableKern
             return super.getText(element);
         }
     }
-    
+
     /**
-     * Check state listener for the tracepoints tree. 
+     * Check state listener for the tracepoints tree.
      */
-    final public class KernelCheckListener implements ICheckStateListener {
+    public final class KernelCheckListener implements ICheckStateListener {
         @Override
         public void checkStateChanged(CheckStateChangedEvent event) {
           if (event.getChecked()) {
               if (event.getElement() instanceof KernelProviderComponent) {
                   fTracepointsViewer.setSubtreeChecked(event.getElement(), true);
-              } 
-          } else { 
+              }
+          } else {
               if (event.getElement() instanceof KernelProviderComponent) {
                   fTracepointsViewer.setSubtreeChecked(event.getElement(), false);
               } else {
This page took 0.029844 seconds and 5 git commands to generate.