tmf/lttng: Fix newly-introduced Javadoc warnings
authorAlexandre Montplaisir <alexmonthy@voxpopuli.im>
Thu, 1 Nov 2012 18:28:35 +0000 (14:28 -0400)
committerAlexandre Montplaisir <alexmonthy@voxpopuli.im>
Mon, 5 Nov 2012 22:26:13 +0000 (17:26 -0500)
Change-Id: Ic0562e0dbc8538603f9749c8dfe402b5cf34fc01
Signed-off-by: Alexandre Montplaisir <alexmonthy@voxpopuli.im>
Reviewed-on: https://git.eclipse.org/r/8474

18 files changed:
org.eclipse.linuxtools.ctf.core/src/org/eclipse/linuxtools/ctf/core/trace/CTFTraceReader.java
org.eclipse.linuxtools.lttng2.kernel.ui/src/org/eclipse/linuxtools/internal/lttng2/kernel/ui/viewers/events/LTTng2EventsTable.java
org.eclipse.linuxtools.lttng2.ui.tests/stubs/org/eclipse/linuxtools/internal/lttng2/stubs/shells/TestCommandShell.java
org.eclipse.linuxtools.lttng2.ui/src/org/eclipse/linuxtools/internal/lttng2/ui/views/control/handlers/ChangeChannelStateHandler.java
org.eclipse.linuxtools.lttng2.ui/src/org/eclipse/linuxtools/internal/lttng2/ui/views/control/handlers/ChangeEventStateHandler.java
org.eclipse.linuxtools.lttng2.ui/src/org/eclipse/linuxtools/internal/lttng2/ui/views/control/handlers/ImportHandler.java
org.eclipse.linuxtools.lttng2.ui/src/org/eclipse/linuxtools/internal/lttng2/ui/views/control/service/LTTngControlService.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/internal/tmf/core/statesystem/historytree/HTNode.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/internal/tmf/core/statesystem/historytree/HistoryTreeBackend.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/component/TmfDataProvider.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/ctfadaptor/CtfTmfEventField.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/ctfadaptor/CtfTmfTrace.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/event/TmfTimestampFormat.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/request/TmfDataRequest.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/statesystem/AbstractStateChangeInput.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/statistics/TmfStateStatistics.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/trace/TmfCheckpointIndexer.java
org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/tmf/core/trace/TmfTrace.java

index f705cac1425a073e78ff6eceb3a34d6fff3d16f2..70626ed8b6826fa67d5b28ec1bbb5c5291070df0 100644 (file)
@@ -66,10 +66,6 @@ public class CTFTraceReader {
      */
     private long endTime;
 
-    protected void setEndTime(long endTime) {
-        this.endTime = endTime;
-    }
-
     // ------------------------------------------------------------------------
     // Constructors
     // ------------------------------------------------------------------------
@@ -132,6 +128,17 @@ public class CTFTraceReader {
         return this.startTime;
     }
 
+    /**
+     * Set the trace's end time
+     *
+     * @param endTime
+     *            The end time to use
+     */
+    protected void setEndTime(long endTime) {
+        this.endTime = endTime;
+    }
+
+
     // ------------------------------------------------------------------------
     // Operations
     // ------------------------------------------------------------------------
index f0d1671b54be80713a111b8204a8548d0eee881c..a2420bd1d4a425f52eaa441ee2572e7e0b755660 100644 (file)
@@ -68,10 +68,6 @@ public class LTTng2EventsTable extends TmfEventsTable {
         fTable.getColumns()[3].setData(Key.FIELD_ID, ITmfEvent.EVENT_FIELD_CONTENT);
     }
 
-    /**
-     * @param event
-     * @return
-     */
     @Override
     protected ITmfEventField[] extractItemFields(ITmfEvent event) {
         ITmfEventField[] fields = new TmfEventField[0];
index 0aa5c1f6d4823c0c856459b5d1ddeec61584825f..82537cc8cda40a47b3d4ba25167b157584757c34 100644 (file)
@@ -22,6 +22,7 @@ import org.eclipse.linuxtools.internal.lttng2.ui.views.control.remote.ICommandSh
  */
 public class TestCommandShell implements ICommandShell {
 
+    /** If the shell is connected */
     protected boolean fIsConnected = false;
 
     @Override
index 7767489920e341f2e06d65f531f265459572ff0e..742220a658063003b4b7cce55d0a3cbe022cc8e8 100644 (file)
@@ -67,7 +67,7 @@ abstract public class ChangeChannelStateHandler extends BaseControlViewHandler {
      * @param domain - the domain of the channels.
      * @param channelNames - a list of channel names
      * @param monitor - a progress monitor
-     * @throws ExecutionException
+     * @throws ExecutionException If the command fails
      */
     abstract protected void changeState(TraceDomainComponent domain, List<String> channelNames, IProgressMonitor monitor) throws ExecutionException;
 
index d6a6f17011f8d509a9c13211c5e4fc6457277c1b..97639212a036c6e022d1f55bd001ac9388f10028 100644 (file)
@@ -67,7 +67,7 @@ abstract public class ChangeEventStateHandler extends BaseControlViewHandler {
      * @param channel - channel of events to be enabled
      * @param eventNames - list event names
      * @param monitor - a progress monitor
-     * @throws ExecutionException
+     * @throws ExecutionException If the command fails
      */
     abstract protected void changeState(TraceChannelComponent channel, List<String> eventNames, IProgressMonitor monitor) throws ExecutionException;
 
index 4e5cc568196bf6e7e4b4fc7a9f72803c24a2d5cd..44bc336c7b72f76a9a4c3a06e55f52c343c2ccd7 100644 (file)
@@ -56,6 +56,10 @@ public class ImportHandler extends BaseControlViewHandler {
     // ------------------------------------------------------------------------
     // Attributes
     // ------------------------------------------------------------------------
+
+    /**
+     * The command parameter
+     */
     protected CommandParameter fParam;
 
     // ------------------------------------------------------------------------
index 5718b9ba36f8baaa5988bb5b31891a17d9a13762..3a3b375731e6d3c1509cf9329efbe330804143c1 100644 (file)
@@ -1169,6 +1169,7 @@ public class LTTngControlService implements ILttngControlService {
      *            - a progress monitor
      * @return the command result
      * @throws ExecutionException
+     *             If the command fails
      */
     protected ICommandResult executeCommand(String command,
             IProgressMonitor monitor) throws ExecutionException {
index b0cc54d936ad84c56d1194c945ca9e808eff2aea..12b10724c0f3f81a50058859ecb17bcfa08c4d3a 100644 (file)
@@ -2,12 +2,12 @@
  * Copyright (c) 2012 Ericsson
  * Copyright (c) 2010, 2011 École Polytechnique de Montréal
  * Copyright (c) 2010, 2011 Alexandre Montplaisir <alexandre.montplaisir@gmail.com>
- * 
+ *
  * 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
- * 
+ *
  *******************************************************************************/
 
 package org.eclipse.linuxtools.internal.tmf.core.statesystem.historytree;
@@ -27,9 +27,9 @@ import org.eclipse.linuxtools.tmf.core.statevalue.TmfStateValue;
 
 /**
  * The base class for all the types of nodes that go in the History Tree.
- * 
+ *
  * @author alexmont
- * 
+ *
  */
 abstract class HTNode {
 
@@ -67,7 +67,7 @@ abstract class HTNode {
     /**
      * Reader factory constructor. Build a Node object (of the right type) by
      * reading a block in the file.
-     * 
+     *
      * @param tree
      *            Reference to the HT which will own this node
      * @param fc
@@ -232,7 +232,7 @@ abstract class HTNode {
 
     /**
      * Add an interval to this node
-     * 
+     *
      * @param newInterval
      */
     void addInterval(HTInterval newInterval) {
@@ -248,7 +248,7 @@ abstract class HTNode {
     /**
      * We've received word from the containerTree that newest nodes now exist to
      * our right. (Puts isDone = true and sets the endtime)
-     * 
+     *
      * @param endtime
      *            The nodeEnd time that the node will have
      * @throws TimeRangeException
@@ -286,7 +286,7 @@ abstract class HTNode {
      * The method to fill up the stateInfo (passed on from the Current State
      * Tree when it does a query on the SHT). We'll replace the data in that
      * vector with whatever relevant we can find from this node
-     * 
+     *
      * @param stateInfo
      *            The same stateInfo that comes from SHT's doQuery()
      * @param t
@@ -319,7 +319,7 @@ abstract class HTNode {
     /**
      * Get a single Interval from the information in this node If the
      * key/timestamp pair cannot be found, we return null.
-     * 
+     *
      * @param key
      * @param t
      * @return The Interval containing the information we want, or null if it
@@ -471,7 +471,7 @@ abstract class HTNode {
 
     /**
      * Debugging function that prints out the contents of this node
-     * 
+     *
      * @param writer
      *            PrintWriter in which we will print the debug output
      */
@@ -505,20 +505,20 @@ abstract class HTNode {
     final static int getCommonHeaderSize() {
         /*
          * 1 - byte (type)
-         * 
+         *
          * 16 - 2x long (start time, end time)
-         * 
+         *
          * 16 - 4x int (seq number, parent seq number, intervalcount, strings
          * section pos.)
-         * 
+         *
          * 1 - byte (done or not)
          */
         return 34;
     }
 
-    /**
-     * @name Abstract methods
-     */
+    // ------------------------------------------------------------------------
+    // Abstract methods
+    // ------------------------------------------------------------------------
 
     protected abstract byte getNodeType();
 
index e0116a9ec9d1165b10dcc1bb78aa6931e63eb1ee..d794831301ba95436a70a276fe22ca9405039d9d 100644 (file)
@@ -35,7 +35,10 @@ import org.eclipse.linuxtools.tmf.core.statevalue.TmfStateValue;
  */
 public class HistoryTreeBackend implements IStateHistoryBackend {
 
+    /** The history tree that sits underneath */
     protected final HistoryTree sht;
+
+    /** Direct reference to the tree's IO object */
     private final HT_IO treeIO;
 
     /** Indicates if the history tree construction is done */
index 2252a8bfb8f58aff59e2e90a13ce7225f583df25..0341ef56c576803e6887c530b27dcc0aa81df3c7 100644 (file)
@@ -61,12 +61,21 @@ public abstract class TmfDataProvider extends TmfComponent implements ITmfDataPr
     // Attributes
     // ------------------------------------------------------------------------
 
+    /** The type of event handled by this provider */
     protected Class<? extends ITmfEvent> fType;
+
+    /** Is there some data being logged? */
     protected boolean fLogData;
+
+    /** Are errors being logged? */
     protected boolean fLogError;
 
-    protected int fQueueSize = DEFAULT_QUEUE_SIZE;
+    /** Queue of events */
     protected BlockingQueue<ITmfEvent> fDataQueue;
+
+    /** Size of the fDataQueue */
+    protected int fQueueSize = DEFAULT_QUEUE_SIZE;
+
     private TmfRequestExecutor fExecutor;
 
     private int fSignalDepth = 0;
@@ -110,6 +119,16 @@ public abstract class TmfDataProvider extends TmfComponent implements ITmfDataPr
         TmfProviderManager.register(fType, this);
     }
 
+    /**
+     * Constructor specifying the event type and the queue size.
+     *
+     * @param name
+     *            Name of the provider
+     * @param type
+     *            Type of event that will be handled
+     * @param queueSize
+     *            Size of the event queue
+     */
     protected TmfDataProvider(String name, Class<? extends ITmfEvent> type, int queueSize) {
         this();
         fQueueSize = queueSize;
@@ -232,8 +251,16 @@ public abstract class TmfDataProvider extends TmfComponent implements ITmfDataPr
     // Coalescing (primitive test...)
     // ------------------------------------------------------------------------
 
+    /** List of coalesced requests */
     protected Vector<TmfCoalescedDataRequest> fPendingCoalescedRequests = new Vector<TmfCoalescedDataRequest>();
 
+    /**
+     * Create a new request from an existing one, and add it to the coalesced
+     * requests
+     *
+     * @param request
+     *            The request to copy
+     */
     protected void newCoalescedDataRequest(ITmfDataRequest request) {
         synchronized (fLock) {
             TmfCoalescedDataRequest coalescedRequest = new TmfCoalescedDataRequest(request.getDataType(), request.getIndex(),
@@ -247,6 +274,12 @@ public abstract class TmfDataProvider extends TmfComponent implements ITmfDataPr
         }
     }
 
+    /**
+     * Add an existing requests to the list of coalesced ones
+     *
+     * @param request
+     *            The request to add to the list
+     */
     protected void coalesceDataRequest(ITmfDataRequest request) {
         synchronized (fLock) {
             for (TmfCoalescedDataRequest coalescedRequest : fPendingCoalescedRequests) {
@@ -275,6 +308,12 @@ public abstract class TmfDataProvider extends TmfComponent implements ITmfDataPr
         }
     }
 
+    /**
+     * Queue a request.
+     *
+     * @param request
+     *            The data request
+     */
     protected void queueRequest(final ITmfDataRequest request) {
 
         if (fExecutor.isShutdown()) {
@@ -355,10 +394,20 @@ public abstract class TmfDataProvider extends TmfComponent implements ITmfDataPr
             TmfCoreTracer.traceRequest(request, "QUEUED"); //$NON-NLS-1$
         }
         fExecutor.execute(thread);
-
     }
 
-    protected void queueBackgroundRequest(final ITmfDataRequest request, final int blockSize, final boolean indexing) {
+    /**
+     * Queue a background request
+     *
+     * @param request
+     *            The request
+     * @param blockSize
+     *            The request should be split in chunks of this size
+     * @param indexing
+     *            Should we index the chunks
+     */
+    protected void queueBackgroundRequest(final ITmfDataRequest request,
+            final int blockSize, final boolean indexing) {
 
         final TmfDataProvider provider = this;
 
@@ -445,7 +494,9 @@ public abstract class TmfDataProvider extends TmfComponent implements ITmfDataPr
      * specific and will be updated by getNext().
      *
      * @param request
-     * @return an application specific context; null if request can't be serviced
+     *            The request
+     * @return Sn application specific context; null if request can't be
+     *         serviced
      */
     protected abstract ITmfContext armRequest(ITmfDataRequest request);
 
index 177654b049530ba9fefbaf0b44415f200382f027..4d7918e9283b676a17120c74f959bb3a3d392a90 100644 (file)
@@ -59,6 +59,7 @@ public abstract class CtfTmfEventField implements ITmfEventField {
     // Attributes
     // ------------------------------------------------------------------------
 
+    /** The name of this field */
     protected final String name;
 
     // ------------------------------------------------------------------------
index e08c0cc40d55968bfd3d11849a2e9cf516014e5b..abb586bedb930ce60806a83c5f0e3cc9e7523407 100644 (file)
@@ -265,10 +265,13 @@ public class CtfTmfTrace extends TmfTrace implements ITmfEventParser {
     }
 
     /**
+     * Build the state system(s) associated with this trace type.
+     *
      * Suppressing the warning, because the 'throws' will usually happen in
      * sub-classes.
      *
      * @throws TmfTraceException
+     *             If there is a problem during the build
      */
     @SuppressWarnings("unused")
     protected void buildStateSystem() throws TmfTraceException {
index dac4ee3a5dffd4966ac437ddf92b2f4511c23e97..e6661fb226420eac7fb195e0de1894e809f6739a 100644 (file)
@@ -235,12 +235,15 @@ public class TmfTimestampFormat extends SimpleDateFormat {
      */
     protected String fSupplPatternLetters = "TSCN"; //$NON-NLS-1$
 
-    /**
+    /*
      * The bracketing symbols used to mitigate the risk of a format string
      * that contains escaped sequences that would conflict with our format
      * extension.
      */
+    /** The open bracket symbol */
     protected String fOpenBracket   = "[&"; //$NON-NLS-1$
+
+    /** The closing bracket symbol */
     protected String fCloseBracket  = "&]"; //$NON-NLS-1$
 
     // ------------------------------------------------------------------------
index e7abfb8b25f32e0867cd9d68f0bfd55355f4626f..025657ab0d1d66d8bd55ddf282df10d893f976f8 100644 (file)
@@ -90,11 +90,21 @@ public abstract class TmfDataRequest implements ITmfDataRequest {
 
     private final Class<? extends ITmfEvent> fDataType;
     private final ExecutionType fExecType;
-    private final int fRequestId; // A unique request ID
-    protected long fIndex; // The index (rank) of the requested event
-    protected int fNbRequested; // The number of requested events (ALL_DATA for all)
-    private final int fBlockSize; // The block size (for BG requests)
-    private int fNbRead; // The number of reads so far
+
+    /** A unique request ID */
+    private final int fRequestId;
+
+    /** The index (rank) of the requested event */
+    protected long fIndex;
+
+    /** The number of requested events (ALL_DATA for all) */
+    protected int fNbRequested;
+
+    /** The block size (for BG requests) */
+    private final int fBlockSize;
+
+    /** The number of reads so far */
+    private int fNbRead;
 
     private final CountDownLatch startedLatch = new CountDownLatch(1);
     private final CountDownLatch completedLatch = new CountDownLatch(1);
index aea775e9a19c5e405eccb4697e06c7b3801dd88f..6ec51b52dfc9a74a01cce558d11bef5f6b0a5b85 100644 (file)
@@ -43,9 +43,11 @@ public abstract class AbstractStateChangeInput implements IStateChangeInput {
     private final Thread eventHandlerThread;
 
     private boolean ssAssigned;
-    protected ITmfStateSystemBuilder ss;
     private ITmfEvent currentEvent;
 
+    /** State system in which to insert the state changes */
+    protected ITmfStateSystemBuilder ss;
+
     /**
      * Instantiate a new state provider plugin.
      *
index 8b9ff6ed51c5e40583ea8e041a5fcd3ca044415b..7eeff57631f92862c3b0c2517f1fbf9d1c42076e 100644 (file)
@@ -330,7 +330,7 @@ public class TmfStateStatistics implements ITmfStatistics {
         return map;
     }
 
-    protected long checkStartTime(ITmfTimestamp startTs) {
+    private long checkStartTime(ITmfTimestamp startTs) {
         long start = startTs.normalize(0, ITmfTimestamp.NANOSECOND_SCALE).getValue();
         if (start < stats.getStartTime()) {
             return stats.getStartTime();
@@ -338,7 +338,7 @@ public class TmfStateStatistics implements ITmfStatistics {
         return start;
     }
 
-    protected long checkEndTime(ITmfTimestamp endTs) {
+    private long checkEndTime(ITmfTimestamp endTs) {
         long end = endTs.normalize(0, ITmfTimestamp.NANOSECOND_SCALE).getValue();
         if (end > stats.getCurrentEndTime()) {
             return stats.getCurrentEndTime();
index 3e212807eee3fec13f74c98c7dcdcb0f8bfa3947..1861403be972fd9cbea5261b147feb06eb64e3d5 100644 (file)
@@ -56,13 +56,13 @@ public class TmfCheckpointIndexer implements ITmfTraceIndexer {
     // Attributes
     // ------------------------------------------------------------------------
 
-    // The event trace to index
+    /** The event trace to index */
     protected final ITmfTrace fTrace;
 
-    // The interval between checkpoints
+    /** The interval between checkpoints */
     private final int fCheckpointInterval;
 
-    // The event trace to index
+    /** The event trace to index */
     private boolean fIsIndexing;
 
     /**
index d8b347cb451859613b7932a7232d483174f1d4a3..ea18b442fd081056e97b60f3b34293dd22f89a34 100644 (file)
@@ -109,7 +109,7 @@ public abstract class TmfTrace extends TmfEventProvider implements ITmfTrace {
      * @param type the trace event type
      * @param path the trace path
      * @param cacheSize the trace cache size
-     * @throws TmfTraceException
+     * @throws TmfTraceException If something failed during the opening
      */
     protected TmfTrace(final IResource resource, final Class<? extends ITmfEvent> type, final String path, final int cacheSize) throws TmfTraceException {
         this(resource, type, path, cacheSize, 0);
@@ -124,7 +124,7 @@ public abstract class TmfTrace extends TmfEventProvider implements ITmfTrace {
      * @param path the trace path
      * @param cacheSize the trace cache size
      * @param interval the trace streaming interval
-     * @throws TmfTraceException
+     * @throws TmfTraceException If something failed during the opening
      */
     protected TmfTrace(final IResource resource, final Class<? extends ITmfEvent> type, final String path, final int cacheSize, final long interval) throws TmfTraceException {
         this(resource, type, path, cacheSize, interval, null);
@@ -138,8 +138,9 @@ public abstract class TmfTrace extends TmfEventProvider implements ITmfTrace {
      * @param type the trace event type
      * @param path the trace path
      * @param cacheSize the trace cache size
+     * @param interval the trace streaming interval
      * @param indexer the trace indexer
-     * @throws TmfTraceException
+     * @throws TmfTraceException If something failed during the opening
      */
     protected TmfTrace(final IResource resource, final Class<? extends ITmfEvent> type, final String path, final int cacheSize,
             final long interval, final ITmfTraceIndexer indexer) throws TmfTraceException {
@@ -153,9 +154,10 @@ public abstract class TmfTrace extends TmfEventProvider implements ITmfTrace {
      * @param type the trace event type
      * @param path the trace path
      * @param cacheSize the trace cache size
+     * @param interval the trace streaming interval
      * @param indexer the trace indexer
      * @param parser the trace event parser
-     * @throws TmfTraceException
+     * @throws TmfTraceException If something failed during the opening
      */
     protected TmfTrace(final IResource resource, final Class<? extends ITmfEvent> type, final String path, final int cacheSize,
             final long interval, final ITmfTraceIndexer indexer, final ITmfEventParser parser) throws TmfTraceException {
@@ -205,7 +207,7 @@ public abstract class TmfTrace extends TmfEventProvider implements ITmfTrace {
      * @param path the trace path
      * @param type the trace event type
      *
-     * @throws TmfTraceException
+     * @throws TmfTraceException If something failed during the initialization
      */
     protected void initialize(final IResource resource, final String path, final Class<? extends ITmfEvent> type) throws TmfTraceException {
         if (path == null) {
@@ -252,9 +254,11 @@ public abstract class TmfTrace extends TmfEventProvider implements ITmfTrace {
     }
 
     /**
-     * The default implementation of TmfTrace uses a TmfStatistics backend.
+     * The default implementation of TmfTrace uses a TmfStatistics back-end.
      * Override this if you want to specify another type (or none at all).
      *
+     * @throws TmfTraceException
+     *             If there was a problem setting up the statistics
      * @since 2.0
      */
     protected void buildStatistics() throws TmfTraceException {
This page took 0.035773 seconds and 5 git commands to generate.