From 4a9c1f0772f01505c9fb1bc31a93a081f0313bc4 Mon Sep 17 00:00:00 2001 From: Alexandre Montplaisir Date: Fri, 15 Mar 2013 17:33:54 -0400 Subject: [PATCH] ctf: Disable NLS warnings in test plugin Test plugins are never meant to be shipped to end-users, so instead of having to put $NON-NLS-1$ and SuppressWarnings everywhere, just disable those in the project settings instead. Change-Id: Ifc9d23e4ce9944fb03cb8c019dd5176d2247a70e Signed-off-by: Alexandre Montplaisir Reviewed-on: https://git.eclipse.org/r/11214 Tested-by: Hudson CI Reviewed-by: Bernd Hufmann IP-Clean: Bernd Hufmann --- .../.settings/org.eclipse.jdt.core.prefs | 2 +- .../ctf/core/tests/shared/CtfTestTraces.java | 1 - .../ctf/core/tests/CtfCorePluginTest.java | 6 +- .../ctf/core/tests/CtfCoreTestPlugin.java | 2 +- .../ctf/core/tests/event/CTFCallsiteTest.java | 3 +- .../core/tests/event/CTFEventFieldTest.java | 12 ++-- .../ctf/core/tests/headless/ReadTrace.java | 3 +- .../ctf/core/tests/io/BitBufferTest.java | 7 +-- .../CTFTraceCallsitePerformanceTest.java | 5 +- .../core/tests/trace/CTFTraceReaderTest.java | 4 +- .../ctf/core/tests/trace/CTFTraceTest.java | 1 - .../ctf/core/tests/trace/IOstructgenTest.java | 5 +- .../StreamInputPacketIndexEntryTest.java | 8 +-- .../tests/trace/StreamInputReaderTest.java | 4 +- .../ctf/core/tests/trace/StreamInputTest.java | 4 +- .../ctf/core/tests/trace/UtilsTest.java | 4 +- .../tests/types/ArrayDeclarationTest.java | 6 +- .../core/tests/types/ArrayDefinitionTest.java | 29 ++++------ .../ctf/core/tests/types/DefinitionTest.java | 2 +- .../core/tests/types/EnumDeclarationTest.java | 8 +-- .../core/tests/types/EnumDefinitionTest.java | 8 +-- .../tests/types/EventDeclarationTest.java | 12 ++-- .../tests/types/FloatDeclarationTest.java | 2 +- .../core/tests/types/FloatDefinitionTest.java | 16 +++-- .../tests/types/IntegerDeclarationTest.java | 10 ++-- .../tests/types/IntegerDefinitionTest.java | 7 ++- .../tests/types/SequenceDeclarationTest.java | 16 +++-- .../tests/types/SequenceDefinitionTest.java | 12 ++-- .../tests/types/StringDeclarationTest.java | 12 ++-- .../tests/types/StringDefinitionTest.java | 4 +- .../tests/types/StructDeclarationTest.java | 10 ++-- .../tests/types/StructDefinitionTest.java | 39 +++++-------- .../tests/types/VariantDeclarationTest.java | 37 ++++++------ .../tests/types/VariantDefinitionTest.java | 58 +++++++++---------- 34 files changed, 162 insertions(+), 197 deletions(-) diff --git a/org.eclipse.linuxtools.ctf.core.tests/.settings/org.eclipse.jdt.core.prefs b/org.eclipse.linuxtools.ctf.core.tests/.settings/org.eclipse.jdt.core.prefs index bca8c277d1..2e1d0e670c 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/.settings/org.eclipse.jdt.core.prefs +++ b/org.eclipse.linuxtools.ctf.core.tests/.settings/org.eclipse.jdt.core.prefs @@ -60,7 +60,7 @@ org.eclipse.jdt.core.compiler.problem.missingSerialVersion=warning org.eclipse.jdt.core.compiler.problem.missingSynchronizedOnInheritedMethod=warning org.eclipse.jdt.core.compiler.problem.noEffectAssignment=warning org.eclipse.jdt.core.compiler.problem.noImplicitStringConversion=warning -org.eclipse.jdt.core.compiler.problem.nonExternalizedStringLiteral=warning +org.eclipse.jdt.core.compiler.problem.nonExternalizedStringLiteral=ignore org.eclipse.jdt.core.compiler.problem.nullAnnotationInferenceConflict=warning org.eclipse.jdt.core.compiler.problem.nullReference=error org.eclipse.jdt.core.compiler.problem.nullSpecViolation=error diff --git a/org.eclipse.linuxtools.ctf.core.tests/shared/org/eclipse/linuxtools/ctf/core/tests/shared/CtfTestTraces.java b/org.eclipse.linuxtools.ctf.core.tests/shared/org/eclipse/linuxtools/ctf/core/tests/shared/CtfTestTraces.java index ca27fcc533..71ce0bf1c9 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/shared/org/eclipse/linuxtools/ctf/core/tests/shared/CtfTestTraces.java +++ b/org.eclipse.linuxtools.ctf.core.tests/shared/org/eclipse/linuxtools/ctf/core/tests/shared/CtfTestTraces.java @@ -21,7 +21,6 @@ import org.eclipse.linuxtools.ctf.core.trace.CTFTrace; * * @author alexmont */ -@SuppressWarnings("nls") public abstract class CtfTestTraces { /* diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/CtfCorePluginTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/CtfCorePluginTest.java index 6220816371..2988773395 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/CtfCorePluginTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/CtfCorePluginTest.java @@ -41,19 +41,19 @@ public class CtfCorePluginTest { @Test public void testCtfCorePluginId() { assertEquals( - "Plugin ID", "org.eclipse.linuxtools.ctf", Activator.PLUGIN_ID); //$NON-NLS-1$ //$NON-NLS-2$ + "Plugin ID", "org.eclipse.linuxtools.ctf", Activator.PLUGIN_ID); } @Test public void testGetDefault() { Activator plugin = Activator.getDefault(); - assertEquals("getDefault()", plugin, fPlugin); //$NON-NLS-1$ + assertEquals("getDefault()", plugin, fPlugin); } @Test public void testLog() { try { - Activator.getDefault().log("Some message"); //$NON-NLS-1$ + Activator.getDefault().log("Some message"); } catch (Exception e) { fail(); } diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/CtfCoreTestPlugin.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/CtfCoreTestPlugin.java index 917f8956a1..40ee63ab36 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/CtfCoreTestPlugin.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/CtfCoreTestPlugin.java @@ -25,7 +25,7 @@ public class CtfCoreTestPlugin extends Plugin { // ------------------------------------------------------------------------ /** The plug-in ID */ - public static final String PLUGIN_ID = "org.eclipse.linuxtools.ctf.core.tests"; //$NON-NLS-1$ + public static final String PLUGIN_ID = "org.eclipse.linuxtools.ctf.core.tests"; // The shared instance private static CtfCoreTestPlugin fPlugin; diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/event/CTFCallsiteTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/event/CTFCallsiteTest.java index 37451940c3..d1aa37a4f5 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/event/CTFCallsiteTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/event/CTFCallsiteTest.java @@ -46,7 +46,6 @@ public class CTFCallsiteTest { // Add additional tear down code here } - @SuppressWarnings("nls") private static CTFCallsite GenerateCS(long ip){ return new CTFCallsite("event name", "func name", ip, "file.java", 1); } @@ -92,7 +91,7 @@ public class CTFCallsiteTest { @Test public void toStringTest(){ CTFCallsite cs = GenerateCS(0x01); - assertEquals("file.java/func name:1", cs.toString()); //$NON-NLS-1$ + assertEquals("file.java/func name:1", cs.toString()); } /** diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/event/CTFEventFieldTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/event/CTFEventFieldTest.java index 3d6e407f4a..66e7d5a952 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/event/CTFEventFieldTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/event/CTFEventFieldTest.java @@ -45,7 +45,7 @@ import org.junit.Test; @SuppressWarnings("javadoc") public class CTFEventFieldTest { - private static final String fieldName = "id"; //$NON-NLS-1$ + private static final String fieldName = "id"; /** @@ -83,7 +83,7 @@ public class CTFEventFieldTest { int len = 32; IntegerDeclaration id = new IntegerDeclaration(len, false, len, ByteOrder.LITTLE_ENDIAN, Encoding.ASCII, null, 32); - String lengthName = "LengthName"; //$NON-NLS-1$ + String lengthName = "LengthName"; StructDeclaration structDec = new StructDeclaration(0); structDec.addField(lengthName, id); StructDefinition structDef = new StructDefinition(structDec, null, @@ -91,7 +91,7 @@ public class CTFEventFieldTest { structDef.lookupInteger(lengthName).setValue(32); SequenceDeclaration sd = new SequenceDeclaration(lengthName, id); - Definition fieldDef = new SequenceDefinition(sd, structDef, "TestX"); //$NON-NLS-1$ + Definition fieldDef = new SequenceDefinition(sd, structDef, "TestX"); ByteBuffer byb = ByteBuffer.allocate(1024); for (int i = 0; i < 1024; i++) { byb.put((byte) i); @@ -134,9 +134,9 @@ public class CTFEventFieldTest { public void testParseField_simple3() { StringDefinition fieldDef = new StringDefinition( new StringDeclaration(), null, fieldName); - fieldDef.setString(new StringBuilder("Hello World")); //$NON-NLS-1$ + fieldDef.setString(new StringBuilder("Hello World")); - String other = "\"Hello World\""; //$NON-NLS-1$ + String other = "\"Hello World\""; assertNotNull(fieldDef); assertEquals(fieldDef.toString(), other); } @@ -187,7 +187,7 @@ public class CTFEventFieldTest { ((IntegerDefinition) ((ArrayDefinition) fieldDef).getDefinitions()[11]).setValue(0); assertNotNull(fieldDef); - String other = "[ 72, 101, 108, 108, 111, 32, 87, 111, 114, 108, 100, 0 ]"; //$NON-NLS-1$ + String other = "[ 72, 101, 108, 108, 111, 32, 87, 111, 114, 108, 100, 0 ]"; assertEquals(other, fieldDef.toString()); } } \ No newline at end of file diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/headless/ReadTrace.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/headless/ReadTrace.java index 9bb6d6478c..739218f9cd 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/headless/ReadTrace.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/headless/ReadTrace.java @@ -30,7 +30,6 @@ public class ReadTrace { /** * @param args */ - @SuppressWarnings("nls") public static void main(String[] args) { final String TRACE_PATH = "traces/kernel"; @@ -108,7 +107,7 @@ public class ReadTrace { */ private static String formatDate(long timestamp) { Date d = new Date(timestamp / 1000000); - DateFormat df = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss."); //$NON-NLS-1$ + DateFormat df = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss."); String output = df.format(d) + (timestamp % 1000000000); return output; } diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/io/BitBufferTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/io/BitBufferTest.java index 286fc28f8d..3265666151 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/io/BitBufferTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/io/BitBufferTest.java @@ -113,8 +113,7 @@ public class BitBufferTest { ByteBuffer result = fixture.getByteBuffer(); assertNotNull(result); - assertEquals( - "java.nio.HeapByteBuffer[pos=0 lim=0 cap=0]", result.toString()); //$NON-NLS-1$ + assertEquals("java.nio.HeapByteBuffer[pos=0 lim=0 cap=0]", result.toString()); assertEquals(false, result.isDirect()); assertEquals(true, result.hasArray()); assertEquals(0, result.arrayOffset()); @@ -134,7 +133,7 @@ public class BitBufferTest { ByteOrder result = fixture.getByteOrder(); assertNotNull(result); - assertEquals("BIG_ENDIAN", result.toString()); //$NON-NLS-1$ + assertEquals("BIG_ENDIAN", result.toString()); } /** @@ -145,7 +144,7 @@ public class BitBufferTest { ByteOrder result = fixture.getByteOrder(); assertNotNull(result); - assertEquals("BIG_ENDIAN", result.toString()); //$NON-NLS-1$ + assertEquals("BIG_ENDIAN", result.toString()); } /** diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/CTFTraceCallsitePerformanceTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/CTFTraceCallsitePerformanceTest.java index 51f6045cf2..c361ec2648 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/CTFTraceCallsitePerformanceTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/CTFTraceCallsitePerformanceTest.java @@ -34,14 +34,12 @@ public class CTFTraceCallsitePerformanceTest { private static final int NUMBER_OF_SEEKS = 100000; - @SuppressWarnings("nls") private final String[] callsites = { "Alligator", "Bunny", "Cat", "Dolphin", "Echidna", "Gazelle", "Heron", "Ibex", "Jackalope", "Koala", "Lynx", "Meerkat", "Narwhal", "Ocelot", "Pangolin", "Quetzal", "Ringtail", "Sandpiper", "Tiger", "Urchin", "Vulture", "Walrus", "X-Ray Tetra", "Zonkey" }; - @SuppressWarnings("nls") private final String[] functions = { "sentence", "together", "children", "mountain", "chipmunk", "crashing", "drinking", "insisted", "insulted", "invented", "squinted", "standing", "swishing", @@ -50,7 +48,6 @@ public class CTFTraceCallsitePerformanceTest { "birthday", "bluebird", "cheerful", "colorful", "daylight", "doghouse", "driveway", "everyone" }; - @SuppressWarnings("nls") private final String[] files = { "Adult.java", "Aeroplane.java", "Air.java", "Airforce.java", "Airport.java", "Album.java", "Alphabet.java", "Apple.java", "Arm.java", "Army.java", "Babby.java" }; @@ -119,7 +116,7 @@ public class CTFTraceCallsitePerformanceTest { private void test(int callsiteSize) { addCallsites(callsiteSize); long ns = testMain(); - System.out.println( "perf ( " + callsiteSize + ", " + ns + ")"); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ + System.out.println( "perf ( " + callsiteSize + ", " + ns + ")"); } private void perfTest() { diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/CTFTraceReaderTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/CTFTraceReaderTest.java index c9ceb3fef9..b236630679 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/CTFTraceReaderTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/CTFTraceReaderTest.java @@ -91,7 +91,7 @@ public class CTFTraceReaderTest { */ @Test(expected = org.eclipse.linuxtools.ctf.core.trace.CTFReaderException.class) public void testOpen_nonexisting() throws CTFReaderException { - CTFTrace trace = new CTFTrace("badfile.bad"); //$NON-NLS-1$ + CTFTrace trace = new CTFTrace("badfile.bad"); CTFTraceReader result = new CTFTraceReader(trace); assertNotNull(result); @@ -105,7 +105,7 @@ public class CTFTraceReaderTest { */ @Test(expected = org.eclipse.linuxtools.ctf.core.trace.CTFReaderException.class) public void testOpen_invalid() throws CTFReaderException { - CTFTrace trace = new CTFTrace(""); //$NON-NLS-1$ + CTFTrace trace = new CTFTrace(""); CTFTraceReader result = new CTFTraceReader(trace); assertNotNull(result); diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/CTFTraceTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/CTFTraceTest.java index 0caf60fd31..3eb133da5c 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/CTFTraceTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/CTFTraceTest.java @@ -41,7 +41,6 @@ import org.junit.Test; * * @author ematkho */ -@SuppressWarnings("nls") public class CTFTraceTest { private static final int TRACE_INDEX = 0; diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/IOstructgenTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/IOstructgenTest.java index 6267df6da7..769a309765 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/IOstructgenTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/IOstructgenTest.java @@ -31,7 +31,6 @@ import org.junit.Test; * * @author Matthew Khouzam */ -@SuppressWarnings("nls") public class IOstructgenTest { @@ -292,7 +291,7 @@ public class IOstructgenTest { deltree(f); } f.mkdirs(); - f = new File(tempTraceDir + "/metadata"); //$NON-NLS-1$ + f = new File(tempTraceDir + "/metadata"); FileWriter fw = new FileWriter(f); fw.write(metadata); fw.close(); @@ -307,7 +306,7 @@ public class IOstructgenTest { for (int i = 0; i < size; i++) { data[i] = 0x00; } - f = new File(tempTraceDir + "/dummyChan"); //$NON-NLS-1$ + f = new File(tempTraceDir + "/dummyChan"); fw = new FileWriter(f); FileOutputStream fos = new FileOutputStream(f); fos.write(magicLE); diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/StreamInputPacketIndexEntryTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/StreamInputPacketIndexEntryTest.java index ac9afbe1c5..f8d9895801 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/StreamInputPacketIndexEntryTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/StreamInputPacketIndexEntryTest.java @@ -61,8 +61,8 @@ public class StreamInputPacketIndexEntryTest { */ @Test public void testStreamInputPacketIndexEntry_1() { - String expectedResult = "StreamInputPacketIndexEntry [offsetBytes=1, " + //$NON-NLS-1$ - "timestampBegin=0, timestampEnd=0]"; //$NON-NLS-1$ + String expectedResult = "StreamInputPacketIndexEntry [offsetBytes=1, " + + "timestampBegin=0, timestampEnd=0]"; assertNotNull(fixture); assertEquals(expectedResult, fixture.toString()); @@ -73,8 +73,8 @@ public class StreamInputPacketIndexEntryTest { */ @Test public void testToString() { - String expectedResult = "StreamInputPacketIndexEntry [offsetBytes=1,"+ //$NON-NLS-1$ - " timestampBegin=1, timestampEnd=1]"; //$NON-NLS-1$ + String expectedResult = "StreamInputPacketIndexEntry [offsetBytes=1,"+ + " timestampBegin=1, timestampEnd=1]"; fixture.setContentSizeBits(1); diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/StreamInputReaderTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/StreamInputReaderTest.java index 7d3fdecd5c..dde9cbd577 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/StreamInputReaderTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/StreamInputReaderTest.java @@ -88,7 +88,7 @@ public class StreamInputReaderTest { * For the tests, we'll use the stream input corresponding to the * CPU 0 */ - if (si.getFilename().endsWith("0_0")) { //$NON-NLS-1$ + if (si.getFilename().endsWith("0_0")) { retVal = new StreamInputReader(si); break; } @@ -114,7 +114,7 @@ public class StreamInputReaderTest { @Test(expected = CTFReaderException.class) public void testStreamInputReader_invalid() throws CTFReaderException { StreamInput streamInput = new StreamInput( - new Stream(new CTFTrace("")), (FileChannel) null, CtfTestTraces.getEmptyFile()); //$NON-NLS-1$ + new Stream(new CTFTrace("")), (FileChannel) null, CtfTestTraces.getEmptyFile()); StreamInputReader result = new StreamInputReader(streamInput); assertNotNull(result); diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/StreamInputTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/StreamInputTest.java index 21373b2da8..396e8892a6 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/StreamInputTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/StreamInputTest.java @@ -77,7 +77,7 @@ public class StreamInputTest { } private static File createFile() { - return new File("Tests/traces/trace20m/channel_0"); //$NON-NLS-1$ + return new File("Tests/traces/trace20m/channel_0"); } /** @@ -143,7 +143,7 @@ public class StreamInputTest { */ @Test public void testLookupDefinition() { - Definition result = fixture.lookupDefinition("id"); //$NON-NLS-1$ + Definition result = fixture.lookupDefinition("id"); assertNull(result); } diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/UtilsTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/UtilsTest.java index abe1df5d45..8a015eb40c 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/UtilsTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/trace/UtilsTest.java @@ -94,7 +94,7 @@ public class UtilsTest { assertNotNull(result); assertEquals(72339069014638592L, result.getLeastSignificantBits()); assertEquals(72339069014638592L, result.getMostSignificantBits()); - assertEquals("01010000-0000-0000-0101-000000000000", result.toString()); //$NON-NLS-1$ + assertEquals("01010000-0000-0000-0101-000000000000", result.toString()); assertEquals(0, result.variant()); assertEquals(0, result.version()); } @@ -113,7 +113,7 @@ public class UtilsTest { assertNotNull(result); assertEquals(0L, result.getLeastSignificantBits()); assertEquals(0L, result.getMostSignificantBits()); - assertEquals("00000000-0000-0000-0000-000000000000", result.toString()); //$NON-NLS-1$ + assertEquals("00000000-0000-0000-0000-000000000000", result.toString()); assertEquals(0, result.variant()); assertEquals(0, result.version()); } diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/ArrayDeclarationTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/ArrayDeclarationTest.java index f60158b0b6..62fa0f1834 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/ArrayDeclarationTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/ArrayDeclarationTest.java @@ -70,7 +70,7 @@ public class ArrayDeclarationTest { ArrayDeclaration result = new ArrayDeclaration(length, elemType); assertNotNull(result); - String left = "[declaration] array["; //$NON-NLS-1$ + String left = "[declaration] array["; String right = result.toString().substring(0, left.length()); assertEquals(left, right); assertEquals(1, result.getLength()); @@ -82,7 +82,7 @@ public class ArrayDeclarationTest { */ @Test public void testCreateDefinition() { - String fieldName = ""; //$NON-NLS-1$ + String fieldName = ""; IDefinitionScope definitionScope = null; ArrayDefinition result; result = fixture.createDefinition(definitionScope, fieldName); @@ -114,7 +114,7 @@ public class ArrayDeclarationTest { @Test public void testToString() { String result = fixture.toString(); - String left = "[declaration] array["; //$NON-NLS-1$ + String left = "[declaration] array["; String right = result.substring(0, left.length()); assertEquals(left, right); diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/ArrayDefinitionTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/ArrayDefinitionTest.java index 5dd0527b7f..f6f17bebba 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/ArrayDefinitionTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/ArrayDefinitionTest.java @@ -72,14 +72,14 @@ public class ArrayDefinitionTest { } private ArrayDefinition createLongArray() { - IntegerDeclaration decl = new IntegerDeclaration(32, false, 10, ByteOrder.BIG_ENDIAN, Encoding.NONE, "none",8); //$NON-NLS-1$ + IntegerDeclaration decl = new IntegerDeclaration(32, false, 10, ByteOrder.BIG_ENDIAN, Encoding.NONE, "none",8); IntegerDefinition[] defs = createIntDefs(10, 32); ArrayDefinition temp = setUpDeclaration(decl, defs); return temp; } private ArrayDefinition createCharArray() { - IntegerDeclaration decl = new IntegerDeclaration(8, false, 10, ByteOrder.BIG_ENDIAN, Encoding.UTF8, "none",8); //$NON-NLS-1$ + IntegerDeclaration decl = new IntegerDeclaration(8, false, 10, ByteOrder.BIG_ENDIAN, Encoding.UTF8, "none",8); IntegerDefinition[] defs = createIntDefs(4,8); ArrayDefinition temp = setUpDeclaration(decl, defs); return temp; @@ -103,7 +103,7 @@ public class ArrayDefinitionTest { private ArrayDefinition setUpDeclaration(IDeclaration decl, Definition[] defs) { ArrayDeclaration ad = new ArrayDeclaration(0, decl); - ArrayDefinition temp = new ArrayDefinition(ad , this.trace , "Testx"); //$NON-NLS-1$ + ArrayDefinition temp = new ArrayDefinition(ad , this.trace , "Testx"); temp.setDefinitions(defs); return temp; } @@ -116,7 +116,7 @@ public class ArrayDefinitionTest { IntegerDefinition[] defs = new IntegerDefinition[size]; for (int i = 0; i < size; i++) { - String content = "test" + i; //$NON-NLS-1$ + String content = "test" + i; defs[i] = new IntegerDefinition(new IntegerDeclaration(bits, false, 16, ByteOrder.LITTLE_ENDIAN, Encoding.UTF8, content, 24), null, content); defs[i].setValue(i); @@ -136,7 +136,7 @@ public class ArrayDefinitionTest { StringDefinition[] defs = new StringDefinition[size]; for (int i = 0; i < size; i++) { - String content = "test" + i; //$NON-NLS-1$ + String content = "test" + i; defs[i] = new StringDefinition( new StringDeclaration(Encoding.UTF8), null, content); defs[i].setString(new StringBuilder(content)); @@ -151,11 +151,9 @@ public class ArrayDefinitionTest { @Test public void testArrayDefinition_baseDeclaration() { ArrayDeclaration declaration = charArrayFixture.getDeclaration(); - String fieldName = ""; //$NON-NLS-1$ - - ArrayDefinition result = new ArrayDefinition(declaration, this.trace, - fieldName); + String fieldName = ""; + ArrayDefinition result = new ArrayDefinition(declaration, this.trace, fieldName); assertNotNull(result); } @@ -168,11 +166,9 @@ public class ArrayDefinitionTest { ArrayDeclaration declaration = new ArrayDeclaration(0, new StringDeclaration()); IDefinitionScope definitionScope = null; - String fieldName = ""; //$NON-NLS-1$ - - ArrayDefinition result = new ArrayDefinition(declaration, - definitionScope, fieldName); + String fieldName = ""; + ArrayDefinition result = new ArrayDefinition(declaration, definitionScope, fieldName); assertNotNull(result); } @@ -223,8 +219,6 @@ public class ArrayDefinitionTest { assertFalse(result); } - - /** * Run the boolean isString() method test. */ @@ -233,8 +227,7 @@ public class ArrayDefinitionTest { final IntegerDeclaration id = new IntegerDeclaration(8, false, 16, ByteOrder.LITTLE_ENDIAN, Encoding.UTF8, null, 8); ArrayDeclaration ad = new ArrayDeclaration(0, id); - ArrayDefinition ownFixture = new ArrayDefinition(ad, this.trace, - "Testx"); //$NON-NLS-1$ + ArrayDefinition ownFixture = new ArrayDefinition(ad, this.trace, "Testx"); int size = 4; int bits = 8; @@ -246,8 +239,6 @@ public class ArrayDefinitionTest { assertTrue(result); } - - /** * Run the boolean isString() method test. */ diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/DefinitionTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/DefinitionTest.java index 94b4bd418e..b20c965747 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/DefinitionTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/DefinitionTest.java @@ -82,7 +82,7 @@ public class DefinitionTest { @Test public void testToString() { - Definition fixture = new DefTest(null, "Hello"); //$NON-NLS-1$ + Definition fixture = new DefTest(null, "Hello"); String result = fixture.toString(); assertNotNull(result); diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/EnumDeclarationTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/EnumDeclarationTest.java index 5a8caa8a72..03d8f08b63 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/EnumDeclarationTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/EnumDeclarationTest.java @@ -76,7 +76,7 @@ public class EnumDeclarationTest { EnumDeclaration result = new EnumDeclaration(containerType); assertNotNull(result); - String left = "[declaration] enum["; //$NON-NLS-1$ + String left = "[declaration] enum["; assertEquals(left, result.toString().substring(0, left.length())); } @@ -87,7 +87,7 @@ public class EnumDeclarationTest { public void testAdd() { long low = 1L; long high = 1L; - String label = ""; //$NON-NLS-1$ + String label = ""; boolean result = fixture.add(low, high, label); @@ -101,7 +101,7 @@ public class EnumDeclarationTest { @Test public void testCreateDefinition() { IDefinitionScope definitionScope = null; - String fieldName = ""; //$NON-NLS-1$ + String fieldName = ""; EnumDefinition result = fixture.createDefinition(definitionScope, fieldName); @@ -127,7 +127,7 @@ public class EnumDeclarationTest { public void testToString() { String result = fixture.toString(); - String left = "[declaration] enum["; //$NON-NLS-1$ + String left = "[declaration] enum["; assertEquals(left, result.substring(0, left.length())); } } diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/EnumDefinitionTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/EnumDefinitionTest.java index b8a9552407..f66b6edd90 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/EnumDefinitionTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/EnumDefinitionTest.java @@ -56,9 +56,9 @@ public class EnumDefinitionTest { EnumDeclaration declaration = new EnumDeclaration( new IntegerDeclaration(1, true, 1, ByteOrder.BIG_ENDIAN, Encoding.ASCII, null, 8)); - declaration.add(0, 10, "a"); //$NON-NLS-1$ - declaration.add(11, 20, "b"); //$NON-NLS-1$ - String fieldName = ""; //$NON-NLS-1$ + declaration.add(0, 10, "a"); + declaration.add(11, 20, "b"); + String fieldName = ""; fixture = new EnumDefinition(declaration, null, fieldName); } @@ -131,6 +131,6 @@ public class EnumDefinitionTest { fixture.setIntegerValue(16); String result = fixture.toString(); - assertEquals("{ value = b, container = 16 }", result); //$NON-NLS-1$ + assertEquals("{ value = b, container = 16 }", result); } } \ No newline at end of file diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/EventDeclarationTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/EventDeclarationTest.java index 9e561b626d..7711cd0a85 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/EventDeclarationTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/EventDeclarationTest.java @@ -67,7 +67,7 @@ public class EventDeclarationTest { fixture.setId(1L); fixture.setFields(new StructDeclaration(1L)); fixture.setStream(new Stream(CtfTestTraces.getTestTrace(TRACE_INDEX))); - fixture.setName(""); //$NON-NLS-1$ + fixture.setName(""); } /** @@ -119,7 +119,7 @@ public class EventDeclarationTest { obj.setId(1L); obj.setFields(new StructDeclaration(1L)); obj.setStream(new Stream(CtfTestTraces.getTestTrace(TRACE_INDEX))); - obj.setName(""); //$NON-NLS-1$ + obj.setName(""); assertTrue(fixture.equals(fixture)); boolean result = fixture.equals(obj); @@ -196,7 +196,7 @@ public class EventDeclarationTest { obj.setContext(new StructDeclaration(1L)); obj.setId(1L); obj.setFields(new StructDeclaration(1L)); - obj.setName(""); //$NON-NLS-1$ + obj.setName(""); boolean result = fixture.equals(obj); assertFalse(result); @@ -349,9 +349,9 @@ public class EventDeclarationTest { assertNotNull(ed.getCPU()); assertNotNull(ed.getPacketContext()); assertNotNull(ed.getStreamInputReader()); - assertNull(ed.lookupDefinition("context")); //$NON-NLS-1$ - assertNotNull(ed.lookupDefinition("fields")); //$NON-NLS-1$ - assertNull(ed.lookupDefinition("other")); //$NON-NLS-1$ + assertNull(ed.lookupDefinition("context")); + assertNotNull(ed.lookupDefinition("fields")); + assertNull(ed.lookupDefinition("other")); assertNotNull(ed.toString()); ed.setContext( ed.getFields()); assertNotNull(ed.toString()); diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/FloatDeclarationTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/FloatDeclarationTest.java index bb7ca8879d..c2950cb198 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/FloatDeclarationTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/FloatDeclarationTest.java @@ -45,6 +45,6 @@ public class FloatDeclarationTest { @Test public void toStringTest() { fixture = new FloatDeclaration(8, 24, ByteOrder.nativeOrder(), 0); - assertTrue(fixture.toString().contains("float")); //$NON-NLS-1$ + assertTrue(fixture.toString().contains("float")); } } diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/FloatDefinitionTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/FloatDefinitionTest.java index c7f79a42e4..babd2261b7 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/FloatDefinitionTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/FloatDefinitionTest.java @@ -24,8 +24,6 @@ import org.junit.After; import org.junit.Before; import org.junit.Test; - - /** * The class IntegerDefinitionTest contains tests for the class * {@link IntegerDefinition}. @@ -35,11 +33,13 @@ import org.junit.Test; */ @SuppressWarnings("javadoc") public class FloatDefinitionTest { + private FloatDefinition fixture; private FloatDefinition singleFixture; private FloatDefinition doubleFixture; //all the way. private FloatDeclaration parent; - private static final String fieldName = "float"; //$NON-NLS-1$ + private static final String fieldName = "float"; + /** * Launch the test. * @@ -88,27 +88,25 @@ public class FloatDefinitionTest { @Test public void testFloat32Bit(){ - for(int i = 1; i < 31 ; i++) - { + for(int i = 1; i < 31 ; i++) { parent = new FloatDeclaration(i, 32-i, ByteOrder.nativeOrder(), 0); fixture = parent.createDefinition(null, fieldName); assertNotNull(fixture); fixture.setValue(2.0); - assertTrue(fixture.toString().contains("2")); //$NON-NLS-1$ + assertTrue(fixture.toString().contains("2")); } } @Test public void testFloat64Bit(){ - for(int i = 1; i < 63 ; i++) - { + for(int i = 1; i < 63 ; i++) { parent = new FloatDeclaration(i, 64-i, ByteOrder.nativeOrder(), 0); fixture = parent.createDefinition(null, fieldName); assertNotNull(fixture); BitBuffer input = new BitBuffer(java.nio.ByteBuffer.allocateDirect(128)); fixture.read(input); fixture.setValue(2.0); - assertTrue(fixture.toString().contains("2")); //$NON-NLS-1$ + assertTrue(fixture.toString().contains("2")); } } diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/IntegerDeclarationTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/IntegerDeclarationTest.java index 13cda0db6b..71cd0111c2 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/IntegerDeclarationTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/IntegerDeclarationTest.java @@ -78,7 +78,7 @@ public class IntegerDeclarationTest { assertNotNull(result); assertEquals(1, result.getBase()); assertEquals(false, result.isCharacter()); - String outputValue = "[declaration] integer["; //$NON-NLS-1$ + String outputValue = "[declaration] integer["; assertEquals(outputValue, result.toString().substring(0, outputValue.length())); assertEquals(1, result.getLength()); @@ -101,7 +101,7 @@ public class IntegerDeclarationTest { public void testGetByteOrder() { ByteOrder result = fixture.getByteOrder(); assertNotNull(result); - assertEquals("BIG_ENDIAN", result.toString()); //$NON-NLS-1$ + assertEquals("BIG_ENDIAN", result.toString()); } /** @@ -111,8 +111,8 @@ public class IntegerDeclarationTest { public void testGetEncoding() { Encoding result = fixture.getEncoding(); assertNotNull(result); - assertEquals("ASCII", result.name()); //$NON-NLS-1$ - assertEquals("ASCII", result.toString()); //$NON-NLS-1$ + assertEquals("ASCII", result.name()); + assertEquals("ASCII", result.toString()); assertEquals(1, result.ordinal()); } @@ -175,6 +175,6 @@ public class IntegerDeclarationTest { public void testToString() { String result = fixture.toString(); String trunc = result.substring(0, 22); - assertEquals("[declaration] integer[", trunc); //$NON-NLS-1$ + assertEquals("[declaration] integer[", trunc); } } \ No newline at end of file diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/IntegerDefinitionTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/IntegerDefinitionTest.java index de81eba55e..5863a0a177 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/IntegerDefinitionTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/IntegerDefinitionTest.java @@ -35,8 +35,9 @@ import org.junit.Test; public class IntegerDefinitionTest { private IntegerDefinition fixture; - String name = "testInt"; //$NON-NLS-1$ - String clockName = "clock"; //$NON-NLS-1$ + String name = "testInt"; + String clockName = "clock"; + /** * Launch the test. * @@ -77,7 +78,7 @@ public class IntegerDefinitionTest { IntegerDeclaration declaration = new IntegerDeclaration(1, true, 1, ByteOrder.BIG_ENDIAN, Encoding.ASCII, null, 8); IDefinitionScope definitionScope = null; - String fieldName = ""; //$NON-NLS-1$ + String fieldName = ""; IntegerDefinition result = new IntegerDefinition(declaration, definitionScope, fieldName); diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/SequenceDeclarationTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/SequenceDeclarationTest.java index ffd66b2d03..26923b67a3 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/SequenceDeclarationTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/SequenceDeclarationTest.java @@ -40,7 +40,7 @@ public class SequenceDeclarationTest { private SequenceDeclaration fixture; - static final String fieldName = "LengthName"; //$NON-NLS-1$ + static final String fieldName = "LengthName"; /** * Launch the test. * @@ -66,13 +66,12 @@ public class SequenceDeclarationTest { */ @Test public void testSequenceDeclaration() { - String lengthName = ""; //$NON-NLS-1$ + String lengthName = ""; IDeclaration elemType = new StringDeclaration(); - SequenceDeclaration result = new SequenceDeclaration(lengthName, - elemType); + SequenceDeclaration result = new SequenceDeclaration(lengthName, elemType); assertNotNull(result); - String string = "[declaration] sequence["; //$NON-NLS-1$ + String string = "[declaration] sequence["; assertEquals(string, result.toString().substring(0, string.length())); } @@ -87,11 +86,10 @@ public class SequenceDeclarationTest { StructDeclaration structDec = new StructDeclaration(0); structDec.addField(fieldName, id); - StructDefinition structDef = new StructDefinition(structDec, null, "x"); //$NON-NLS-1$ + StructDefinition structDef = new StructDefinition(structDec, null, "x"); long seqLen = 10; structDef.lookupInteger(fieldName).setValue(seqLen); - SequenceDefinition result = this.fixture.createDefinition(structDef, - fieldName); + SequenceDefinition result = this.fixture.createDefinition(structDef, fieldName); assertNotNull(result); } @@ -110,7 +108,7 @@ public class SequenceDeclarationTest { @Test public void testToString() { String result = fixture.toString(); - String left = "[declaration] sequence["; //$NON-NLS-1$ + String left = "[declaration] sequence["; assertEquals(left, result.substring(0, left.length())); } } diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/SequenceDefinitionTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/SequenceDefinitionTest.java index 763a090970..d0045da7cb 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/SequenceDefinitionTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/SequenceDefinitionTest.java @@ -64,14 +64,14 @@ public class SequenceDefinitionTest { IntegerDeclaration id = new IntegerDeclaration(8, false, 8, ByteOrder.LITTLE_ENDIAN, Encoding.UTF8, null, 8); - String lengthName = "LengthName"; //$NON-NLS-1$ + String lengthName = "LengthName"; structDec = new StructDeclaration(0); structDec.addField(lengthName, id); - structDef = new StructDefinition(structDec, null, "x"); //$NON-NLS-1$ + structDef = new StructDefinition(structDec, null, "x"); structDef.lookupInteger(lengthName).setValue(seqLen); SequenceDeclaration sd = new SequenceDeclaration(lengthName, id); - fixture = new SequenceDefinition(sd, structDef, "TestX"); //$NON-NLS-1$ + fixture = new SequenceDefinition(sd, structDef, "TestX"); BitBuffer input = new BitBuffer( java.nio.ByteBuffer.allocateDirect(seqLen * 8)); for (int i = 0; i < seqLen; i++) { @@ -96,14 +96,14 @@ public class SequenceDefinitionTest { int len = 32; IntegerDeclaration id = new IntegerDeclaration(len, false, len, ByteOrder.LITTLE_ENDIAN, Encoding.UTF8, null,8); - String lengthName = "LengthName"; //$NON-NLS-1$ + String lengthName = "LengthName"; structDec = new StructDeclaration(0); structDec.addField(lengthName, id); - structDef = new StructDefinition(structDec, null, "x"); //$NON-NLS-1$ + structDef = new StructDefinition(structDec, null, "x"); structDef.lookupInteger(lengthName).setValue(seqLen); SequenceDeclaration sd = new SequenceDeclaration(lengthName, id); - SequenceDefinition ret = new SequenceDefinition(sd, structDef, "TestX"); //$NON-NLS-1$ + SequenceDefinition ret = new SequenceDefinition(sd, structDef, "TestX"); BitBuffer input = new BitBuffer( java.nio.ByteBuffer.allocateDirect(seqLen * len)); for (int i = 0; i < seqLen; i++) { diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StringDeclarationTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StringDeclarationTest.java index d03fe9311f..41e0030735 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StringDeclarationTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StringDeclarationTest.java @@ -67,7 +67,7 @@ public class StringDeclarationTest { StringDeclaration result = new StringDeclaration(); assertNotNull(result); - String string = "[declaration] string["; //$NON-NLS-1$ + String string = "[declaration] string["; assertEquals(string, result.toString().substring(0, string.length())); } @@ -80,7 +80,7 @@ public class StringDeclarationTest { StringDeclaration result = new StringDeclaration(encoding); assertNotNull(result); - String string = "[declaration] string["; //$NON-NLS-1$ + String string = "[declaration] string["; assertEquals(string, result.toString().substring(0, string.length())); } @@ -91,7 +91,7 @@ public class StringDeclarationTest { @Test public void testCreateDefinition() { IDefinitionScope definitionScope = null; - String fieldName = "id"; //$NON-NLS-1$ + String fieldName = "id"; StringDefinition result = fixture.createDefinition(definitionScope, fieldName); @@ -106,8 +106,8 @@ public class StringDeclarationTest { Encoding result = fixture.getEncoding(); assertNotNull(result); - assertEquals("ASCII", result.name()); //$NON-NLS-1$ - assertEquals("ASCII", result.toString()); //$NON-NLS-1$ + assertEquals("ASCII", result.name()); + assertEquals("ASCII", result.toString()); assertEquals(1, result.ordinal()); } @@ -126,7 +126,7 @@ public class StringDeclarationTest { @Test public void testToString() { String result = fixture.toString(); - String left = "[declaration] string["; //$NON-NLS-1$ + String left = "[declaration] string["; String right = result.substring(0, left.length()); assertEquals(left, right); diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StringDefinitionTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StringDefinitionTest.java index 79231779ae..065af26e3e 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StringDefinitionTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StringDefinitionTest.java @@ -47,7 +47,7 @@ public class StringDefinitionTest { */ @Before public void setUp() { - String name = "testString"; //$NON-NLS-1$ + String name = "testString"; StringDeclaration stringDec = new StringDeclaration(); fixture = stringDec.createDefinition(null, name); } @@ -68,7 +68,7 @@ public class StringDefinitionTest { public void testStringDefinition() { StringDeclaration declaration = new StringDeclaration(); IDefinitionScope definitionScope = null; - String fieldName = ""; //$NON-NLS-1$ + String fieldName = ""; StringDefinition result = new StringDefinition(declaration, definitionScope, fieldName); diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StructDeclarationTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StructDeclarationTest.java index 159bea3e36..7d8ab256a5 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StructDeclarationTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StructDeclarationTest.java @@ -71,7 +71,7 @@ public class StructDeclarationTest { assertNotNull(fixture); assertEquals(1L, fixture.getMaxAlign()); - String regex = "^\\[declaration\\] struct\\[[0-9a-f]{1,8}\\]$"; //$NON-NLS-1$ + String regex = "^\\[declaration\\] struct\\[[0-9a-f]{1,8}\\]$"; assertTrue(fixture.toString().matches(regex)); } @@ -80,7 +80,7 @@ public class StructDeclarationTest { */ @Test public void testAddField() { - String name = ""; //$NON-NLS-1$ + String name = ""; IDeclaration declaration = new StringDeclaration(); fixture.addField(name, declaration); } @@ -91,7 +91,7 @@ public class StructDeclarationTest { */ @Test public void testCreateDefinition() { - String fieldName = ""; //$NON-NLS-1$ + String fieldName = ""; StructDefinition result = fixture.createDefinition(null, fieldName); assertNotNull(result); } @@ -132,7 +132,7 @@ public class StructDeclarationTest { */ @Test public void testHasField() { - String name = ""; //$NON-NLS-1$ + String name = ""; boolean result = fixture.hasField(name); assertEquals(false, result); @@ -146,6 +146,6 @@ public class StructDeclarationTest { String result = fixture.toString(); String trunc = result.substring(0, 21); - assertEquals("[declaration] struct[", trunc); //$NON-NLS-1$ + assertEquals("[declaration] struct[", trunc); } } diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StructDefinitionTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StructDefinitionTest.java index 6d3813e01e..b45a3b2f21 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StructDefinitionTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/StructDefinitionTest.java @@ -48,31 +48,20 @@ import org.junit.Test; */ public class StructDefinitionTest { - private static final String TEST_STRUCT_ID = "testStruct"; //$NON-NLS-1$ - - private static final String ENUM_2 = "y"; //$NON-NLS-1$ - - private static final String ENUM_1 = "x"; //$NON-NLS-1$ - - private static final String TAG_ID = "Tag"; //$NON-NLS-1$ - - private static final String INT_ID = "_id"; //$NON-NLS-1$ - - private static final String STRING_ID = "_args"; //$NON-NLS-1$ - - private static final String ENUM_ID = "_enumArgs"; //$NON-NLS-1$ - - private static final String SEQUENCE_ID = "_seq"; //$NON-NLS-1$ - - private static final String LENGTH_SEQ = "_len"; //$NON-NLS-1$ + private static final String TEST_STRUCT_ID = "testStruct"; + private static final String ENUM_2 = "y"; + private static final String ENUM_1 = "x"; + private static final String TAG_ID = "Tag"; + private static final String INT_ID = "_id"; + private static final String STRING_ID = "_args"; + private static final String ENUM_ID = "_enumArgs"; + private static final String SEQUENCE_ID = "_seq"; + private static final String LENGTH_SEQ = "_len"; private StructDefinition fixture; - private StructDefinition emptyStruct; - private StructDefinition simpleStruct; - - private static final String VAR_FIELD_NAME = "SomeVariant"; //$NON-NLS-1$ + private static final String VAR_FIELD_NAME = "SomeVariant"; /** * Launch the test. @@ -168,7 +157,7 @@ public class StructDefinitionTest { */ @Test public void testLookupDefinition() { - String lookupPath = "args"; //$NON-NLS-1$ + String lookupPath = "args"; Definition result = fixture.lookupDefinition(lookupPath); assertNotNull(result); @@ -189,7 +178,7 @@ public class StructDefinitionTest { */ @Test public void testLookupInteger_1() { - String name = "_id"; //$NON-NLS-1$ + String name = "_id"; IntegerDefinition result = fixture.lookupInteger(name); assertNotNull(result); } @@ -268,9 +257,9 @@ public class StructDefinitionTest { assertNotNull(result); result = emptyStruct.toString(); - assertEquals("{ }", result); //$NON-NLS-1$ + assertEquals("{ }", result); result = simpleStruct.toString(); - assertEquals("{ _id = 0, _args = \"\" }", result); //$NON-NLS-1$ + assertEquals("{ _id = 0, _args = \"\" }", result); } } \ No newline at end of file diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/VariantDeclarationTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/VariantDeclarationTest.java index 790ce859d2..2f683ed529 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/VariantDeclarationTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/VariantDeclarationTest.java @@ -74,7 +74,7 @@ public class VariantDeclarationTest { public void testVariantDeclaration() { assertNotNull(fixture); assertEquals(false, fixture.isTagged()); - String left = "[declaration] variant["; //$NON-NLS-1$ + String left = "[declaration] variant["; assertEquals(left, fixture.toString().substring(0, left.length())); } @@ -83,8 +83,8 @@ public class VariantDeclarationTest { */ @Test public void testAddField() { - fixture.setTag(""); //$NON-NLS-1$ - String tag = ""; //$NON-NLS-1$ + fixture.setTag(""); + String tag = ""; IDeclaration declaration = new StringDeclaration(); fixture.addField(tag, declaration); } @@ -97,11 +97,10 @@ public class VariantDeclarationTest { */ @Test public void testCreateDefinition() throws CTFReaderException { - fixture.setTag(""); //$NON-NLS-1$ + fixture.setTag(""); IDefinitionScope definitionScope = createDefinitionScope(); - String fieldName = ""; //$NON-NLS-1$ - VariantDefinition result = fixture.createDefinition(definitionScope, - fieldName); + String fieldName = ""; + VariantDefinition result = fixture.createDefinition(definitionScope, fieldName); assertNotNull(result); } @@ -109,14 +108,14 @@ public class VariantDeclarationTest { private static IDefinitionScope createDefinitionScope() throws CTFReaderException { assumeTrue(CtfTestTraces.tracesExist()); VariantDeclaration declaration = new VariantDeclaration(); - declaration.setTag(""); //$NON-NLS-1$ + declaration.setTag(""); VariantDeclaration variantDeclaration = new VariantDeclaration(); - variantDeclaration.setTag(""); //$NON-NLS-1$ + variantDeclaration.setTag(""); VariantDefinition variantDefinition = new VariantDefinition( - variantDeclaration, CtfTestTraces.getTestTrace(TRACE_INDEX), ""); //$NON-NLS-1$ + variantDeclaration, CtfTestTraces.getTestTrace(TRACE_INDEX), ""); IDefinitionScope definitionScope = new StructDefinition( - new StructDeclaration(1L), variantDefinition, ""); //$NON-NLS-1$ - String fieldName = ""; //$NON-NLS-1$ + new StructDeclaration(1L), variantDefinition, ""); + String fieldName = ""; VariantDefinition result = new VariantDefinition(declaration, definitionScope, fieldName); @@ -128,8 +127,8 @@ public class VariantDeclarationTest { */ @Test public void testHasField() { - fixture.setTag(""); //$NON-NLS-1$ - String tag = ""; //$NON-NLS-1$ + fixture.setTag(""); + String tag = ""; boolean result = fixture.hasField(tag); assertEquals(false, result); @@ -140,7 +139,7 @@ public class VariantDeclarationTest { */ @Test public void testIsTagged() { - fixture.setTag(""); //$NON-NLS-1$ + fixture.setTag(""); boolean result = fixture.isTagged(); assertEquals(true, result); @@ -162,8 +161,8 @@ public class VariantDeclarationTest { */ @Test public void testSetTag() { - fixture.setTag(""); //$NON-NLS-1$ - String tag = ""; //$NON-NLS-1$ + fixture.setTag(""); + String tag = ""; fixture.setTag(tag); } @@ -172,9 +171,9 @@ public class VariantDeclarationTest { */ @Test public void testToString() { - fixture.setTag(""); //$NON-NLS-1$ + fixture.setTag(""); String result = fixture.toString(); - String left = "[declaration] variant["; //$NON-NLS-1$ + String left = "[declaration] variant["; String right = result.substring(0, left.length()); assertEquals(left, right); diff --git a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/VariantDefinitionTest.java b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/VariantDefinitionTest.java index 5bea596144..fba1c899e8 100644 --- a/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/VariantDefinitionTest.java +++ b/org.eclipse.linuxtools.ctf.core.tests/src/org/eclipse/linuxtools/ctf/core/tests/types/VariantDefinitionTest.java @@ -60,29 +60,27 @@ public class VariantDefinitionTest { public static void main(String[] args) { new org.junit.runner.JUnitCore().run(VariantDefinitionTest.class); } + StructDefinition structDefinition; - private static final String TEST_STRUCT_ID = "testStruct"; //$NON-NLS-1$ - - private static final String ENUM_7 = "g"; //$NON-NLS-1$ - private static final String ENUM_6 = "f"; //$NON-NLS-1$ - private static final String ENUM_5 = "e"; //$NON-NLS-1$ - private static final String ENUM_4 = "d"; //$NON-NLS-1$ - private static final String ENUM_3 = "c"; //$NON-NLS-1$ - private static final String ENUM_2 = "b"; //$NON-NLS-1$ - private static final String ENUM_1 = "a"; //$NON-NLS-1$ - - private static final String TAG_ID = "a"; //$NON-NLS-1$ - -// private static final String INT_ID = "_id"; //$NON-NLS-1$ -// -// private static final String STRING_ID = "_args"; //$NON-NLS-1$ -// -// private static final String ENUM_ID = "_enumArgs"; //$NON-NLS-1$ -// -// private static final String SEQUENCE_ID = "_seq"; //$NON-NLS-1$ - - private static final String LENGTH_SEQ = "_len"; //$NON-NLS-1$ - private static final String VAR_FIELD_NAME = "var"; //$NON-NLS-1$ + private static final String TEST_STRUCT_ID = "testStruct"; + + private static final String ENUM_7 = "g"; + private static final String ENUM_6 = "f"; + private static final String ENUM_5 = "e"; + private static final String ENUM_4 = "d"; + private static final String ENUM_3 = "c"; + private static final String ENUM_2 = "b"; + private static final String ENUM_1 = "a"; + + private static final String TAG_ID = "a"; + +// private static final String INT_ID = "_id"; +// private static final String STRING_ID = "_args"; +// private static final String ENUM_ID = "_enumArgs"; +// private static final String SEQUENCE_ID = "_seq"; + + private static final String LENGTH_SEQ = "_len"; + private static final String VAR_FIELD_NAME = "var"; private static final String ENUM_8 = null; /** * Perform pre-test initialization. @@ -147,14 +145,14 @@ public class VariantDefinitionTest { @Test public void testVariantDefinition() { VariantDeclaration declaration = new VariantDeclaration(); - declaration.setTag(""); //$NON-NLS-1$ + declaration.setTag(""); VariantDeclaration variantDeclaration = new VariantDeclaration(); - variantDeclaration.setTag(""); //$NON-NLS-1$ + variantDeclaration.setTag(""); VariantDefinition variantDefinition = new VariantDefinition( - variantDeclaration, structDefinition, ""); //$NON-NLS-1$ + variantDeclaration, structDefinition, ""); IDefinitionScope definitionScope = new StructDefinition( - new StructDeclaration(1L), variantDefinition, ""); //$NON-NLS-1$ - String fieldName = ""; //$NON-NLS-1$ + new StructDeclaration(1L), variantDefinition, ""); + String fieldName = ""; VariantDefinition result = new VariantDefinition(declaration, definitionScope, fieldName); @@ -328,7 +326,7 @@ public class VariantDefinitionTest { VariantDefinition vDef; StructDefinition structDef; EnumDefinition tagDefinition; - String fName = ""; //$NON-NLS-1$ + String fName = ""; vDecl = new VariantDeclaration(); vDecl.setTag(fName); @@ -347,10 +345,10 @@ public class VariantDefinitionTest { @Test public void testToString() { String result = fixture.toString(); - assertEquals("{ null = null }", result); //$NON-NLS-1$ + assertEquals("{ null = null }", result); fixture.setCurrentField(ENUM_2); result = fixture.toString(); - assertEquals("{ b = 0 }", result); //$NON-NLS-1$ + assertEquals("{ b = 0 }", result); } } -- 2.34.1