tmf: Fix some Sonar warnings in the state system code

[deliverable/tracecompass.git] / org.eclipse.linuxtools.tmf.core / src / org / eclipse / linuxtools / internal / tmf / core / statesystem / backends / historytree / HT_IO.java

diff --git a/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/internal/tmf/core/statesystem/backends/historytree/HT_IO.java b/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/internal/tmf/core/statesystem/backends/historytree/HT_IO.java
index 7270bd88143c7ba2e0f052eea3a1e0f402fa29cf..c8ff18e148b5de86b56a3cdc40c78a1f15031563 100644 (file)
--- a/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/internal/tmf/core/statesystem/backends/historytree/HT_IO.java
+++ b/org.eclipse.linuxtools.tmf.core/src/org/eclipse/linuxtools/internal/tmf/core/statesystem/backends/historytree/HT_IO.java
@@ -50,15 +50,15 @@ class HT_IO {
      */
     HT_IO(HistoryTree tree, boolean newFile) throws IOException {
         this.tree = tree;
-        historyTreeFile = tree.config.stateFile;
-        boolean success1 = true, success2;
+        historyTreeFile = tree.getConfig().getStateFile();
+        boolean success1 = true;
 
         if (newFile) {
             /* Create a new empty History Tree file */
             if (historyTreeFile.exists()) {
                 success1 = historyTreeFile.delete();
             }
-            success2 = historyTreeFile.createNewFile();
+            boolean success2 = historyTreeFile.createNewFile();
             if (!(success1 && success2)) {
                 /* It seems we do not have permission to create the new file */
                 throw new IOException("Cannot create new file at " + //$NON-NLS-1$
@@ -97,7 +97,7 @@ class HT_IO {
     }
 
     private HTNode readNodeFromMemory(int seqNumber) {
-        for (HTNode node : tree.latestBranch) {
+        for (HTNode node : tree.getLatestBranch()) {
             if (node.getSequenceNumber() == seqNumber) {
                 return node;
             }
@@ -159,12 +159,12 @@ class HT_IO {
     }
 
     File supplyATWriterFile() {
-        return tree.config.stateFile;
+        return tree.getConfig().getStateFile();
     }
 
     long supplyATWriterFilePos() {
-        return HistoryTree.getTreeHeaderSize()
-                + ((long) tree.getNodeCount() * tree.config.blockSize);
+        return HistoryTree.TREE_HEADER_SIZE
+                + ((long) tree.getNodeCount() * tree.getConfig().getBlockSize());
     }
 
     synchronized void closeFile() {
@@ -194,8 +194,8 @@ class HT_IO {
      */
     private void seekFCToNodePos(FileChannel fc, int seqNumber)
             throws IOException {
-        fc.position(HistoryTree.getTreeHeaderSize() + (long) seqNumber
-                * tree.config.blockSize);
+        fc.position(HistoryTree.TREE_HEADER_SIZE + (long) seqNumber
+                * tree.getConfig().getBlockSize());
         /*
          * cast to (long) is needed to make sure the result is a long too and
          * doesn't get truncated