From 48ceaaea2a7deed5b7e5a4dfaf7960c2a6ba555c Mon Sep 17 00:00:00 2001 From: akwizgran <akwizgran@users.sourceforge.net> Date: Fri, 20 Jan 2012 23:28:17 +0000 Subject: [PATCH] Renamed a couple of classes. --- .../sf/briar/transport/ConnectionReaderFactoryImpl.java | 8 +++++--- .../sf/briar/transport/ConnectionWriterFactoryImpl.java | 2 +- ...ilityLayer.java => IncomingReliabilityLayerImpl.java} | 9 +++++---- ...nLayer.java => SegmentedIncomingEncryptionLayer.java} | 4 ++-- ...nLayer.java => SegmentedOutgoingEncryptionLayer.java} | 4 ++-- test/build.xml | 4 ++-- .../net/sf/briar/transport/ConnectionReaderImplTest.java | 3 ++- test/net/sf/briar/transport/FrameReadWriteTest.java | 3 ++- ...st.java => SegmentedIncomingEncryptionLayerTest.java} | 8 ++++---- ...st.java => SegmentedOutgoingEncryptionLayerTest.java} | 8 ++++---- 10 files changed, 29 insertions(+), 24 deletions(-) rename components/net/sf/briar/transport/{NullIncomingReliabilityLayer.java => IncomingReliabilityLayerImpl.java} (77%) rename components/net/sf/briar/transport/{IncomingSegmentedEncryptionLayer.java => SegmentedIncomingEncryptionLayer.java} (96%) rename components/net/sf/briar/transport/{OutgoingSegmentedEncryptionLayer.java => SegmentedOutgoingEncryptionLayer.java} (95%) rename test/net/sf/briar/transport/{IncomingSegmentedEncryptionLayerTest.java => SegmentedIncomingEncryptionLayerTest.java} (96%) rename test/net/sf/briar/transport/{OutgoingSegmentedEncryptionLayerTest.java => SegmentedOutgoingEncryptionLayerTest.java} (95%) diff --git a/components/net/sf/briar/transport/ConnectionReaderFactoryImpl.java b/components/net/sf/briar/transport/ConnectionReaderFactoryImpl.java index a2d73e9a39..a2725ef2d9 100644 --- a/components/net/sf/briar/transport/ConnectionReaderFactoryImpl.java +++ b/components/net/sf/briar/transport/ConnectionReaderFactoryImpl.java @@ -55,7 +55,8 @@ class ConnectionReaderFactoryImpl implements ConnectionReaderFactory { new IncomingAuthenticationLayerImpl(correction, mac, macKey); // No reordering or retransmission IncomingReliabilityLayer reliability = - new NullIncomingReliabilityLayer(authentication); + new IncomingReliabilityLayerImpl(authentication, + new NullFrameWindow()); // Create the reader - don't tolerate errors return new ConnectionReaderImpl(reliability, false); } @@ -81,7 +82,7 @@ class ConnectionReaderFactoryImpl implements ConnectionReaderFactory { Cipher tagCipher = crypto.getTagCipher(); Cipher segCipher = crypto.getSegmentCipher(); IncomingEncryptionLayer encryption = - new IncomingSegmentedEncryptionLayer(in, tagCipher, segCipher, + new SegmentedIncomingEncryptionLayer(in, tagCipher, segCipher, tagKey, segKey, false, bufferedSegment); // No error correction IncomingErrorCorrectionLayer correction = @@ -92,7 +93,8 @@ class ConnectionReaderFactoryImpl implements ConnectionReaderFactory { new IncomingAuthenticationLayerImpl(correction, mac, macKey); // No reordering or retransmission IncomingReliabilityLayer reliability = - new NullIncomingReliabilityLayer(authentication); + new IncomingReliabilityLayerImpl(authentication, + new NullFrameWindow()); // Create the reader - don't tolerate errors return new ConnectionReaderImpl(reliability, false); } diff --git a/components/net/sf/briar/transport/ConnectionWriterFactoryImpl.java b/components/net/sf/briar/transport/ConnectionWriterFactoryImpl.java index 6382955245..404a9bc231 100644 --- a/components/net/sf/briar/transport/ConnectionWriterFactoryImpl.java +++ b/components/net/sf/briar/transport/ConnectionWriterFactoryImpl.java @@ -60,7 +60,7 @@ class ConnectionWriterFactoryImpl implements ConnectionWriterFactory { Cipher tagCipher = crypto.getTagCipher(); Cipher segCipher = crypto.getSegmentCipher(); OutgoingEncryptionLayer encryption = - new OutgoingSegmentedEncryptionLayer(out, capacity, tagCipher, + new SegmentedOutgoingEncryptionLayer(out, capacity, tagCipher, segCipher, tagKey, segKey, false); // No error correction OutgoingErrorCorrectionLayer correction = diff --git a/components/net/sf/briar/transport/NullIncomingReliabilityLayer.java b/components/net/sf/briar/transport/IncomingReliabilityLayerImpl.java similarity index 77% rename from components/net/sf/briar/transport/NullIncomingReliabilityLayer.java rename to components/net/sf/briar/transport/IncomingReliabilityLayerImpl.java index ecd871d22f..a5943a50ab 100644 --- a/components/net/sf/briar/transport/NullIncomingReliabilityLayer.java +++ b/components/net/sf/briar/transport/IncomingReliabilityLayerImpl.java @@ -2,16 +2,17 @@ package net.sf.briar.transport; import java.io.IOException; -class NullIncomingReliabilityLayer implements IncomingReliabilityLayer { +class IncomingReliabilityLayerImpl implements IncomingReliabilityLayer { private final IncomingAuthenticationLayer in; - private final int maxFrameLength; private final FrameWindow window; + private final int maxFrameLength; - NullIncomingReliabilityLayer(IncomingAuthenticationLayer in) { + IncomingReliabilityLayerImpl(IncomingAuthenticationLayer in, + FrameWindow window) { this.in = in; + this.window = window; maxFrameLength = in.getMaxFrameLength(); - window = new NullFrameWindow(); } public boolean readFrame(Frame f) throws IOException, InvalidDataException { diff --git a/components/net/sf/briar/transport/IncomingSegmentedEncryptionLayer.java b/components/net/sf/briar/transport/SegmentedIncomingEncryptionLayer.java similarity index 96% rename from components/net/sf/briar/transport/IncomingSegmentedEncryptionLayer.java rename to components/net/sf/briar/transport/SegmentedIncomingEncryptionLayer.java index 50dacb4485..7f378c0ebb 100644 --- a/components/net/sf/briar/transport/IncomingSegmentedEncryptionLayer.java +++ b/components/net/sf/briar/transport/SegmentedIncomingEncryptionLayer.java @@ -15,7 +15,7 @@ import net.sf.briar.api.crypto.ErasableKey; import net.sf.briar.api.plugins.SegmentSource; import net.sf.briar.api.transport.Segment; -class IncomingSegmentedEncryptionLayer implements IncomingEncryptionLayer { +class SegmentedIncomingEncryptionLayer implements IncomingEncryptionLayer { private final SegmentSource in; private final Cipher tagCipher, segCipher; @@ -29,7 +29,7 @@ class IncomingSegmentedEncryptionLayer implements IncomingEncryptionLayer { private boolean firstSegment = true; private long segmentNumber = 0L; - IncomingSegmentedEncryptionLayer(SegmentSource in, Cipher tagCipher, + SegmentedIncomingEncryptionLayer(SegmentSource in, Cipher tagCipher, Cipher segCipher, ErasableKey tagKey, ErasableKey segKey, boolean tagEverySegment, Segment bufferedSegment) { this.in = in; diff --git a/components/net/sf/briar/transport/OutgoingSegmentedEncryptionLayer.java b/components/net/sf/briar/transport/SegmentedOutgoingEncryptionLayer.java similarity index 95% rename from components/net/sf/briar/transport/OutgoingSegmentedEncryptionLayer.java rename to components/net/sf/briar/transport/SegmentedOutgoingEncryptionLayer.java index eb188ff608..4807e0d09e 100644 --- a/components/net/sf/briar/transport/OutgoingSegmentedEncryptionLayer.java +++ b/components/net/sf/briar/transport/SegmentedOutgoingEncryptionLayer.java @@ -15,7 +15,7 @@ import net.sf.briar.api.crypto.ErasableKey; import net.sf.briar.api.plugins.SegmentSink; import net.sf.briar.api.transport.Segment; -class OutgoingSegmentedEncryptionLayer implements OutgoingEncryptionLayer { +class SegmentedOutgoingEncryptionLayer implements OutgoingEncryptionLayer { private final SegmentSink out; private final Cipher tagCipher, segCipher; @@ -27,7 +27,7 @@ class OutgoingSegmentedEncryptionLayer implements OutgoingEncryptionLayer { private long capacity; - OutgoingSegmentedEncryptionLayer(SegmentSink out, long capacity, + SegmentedOutgoingEncryptionLayer(SegmentSink out, long capacity, Cipher tagCipher, Cipher segCipher, ErasableKey tagKey, ErasableKey segKey, boolean tagEverySegment) { this.out = out; diff --git a/test/build.xml b/test/build.xml index a0c9523ccb..8d73c651a6 100644 --- a/test/build.xml +++ b/test/build.xml @@ -59,9 +59,9 @@ <test name='net.sf.briar.transport.FrameWindowImplTest'/> <test name='net.sf.briar.transport.IncomingEncryptionLayerImplTest'/> <test name='net.sf.briar.transport.IncomingErrorCorrectionLayerImplTest'/> - <test name='net.sf.briar.transport.IncomingSegmentedEncryptionLayerTest'/> <test name='net.sf.briar.transport.OutgoingEncryptionLayerImplTest'/> - <test name='net.sf.briar.transport.OutgoingSegmentedEncryptionLayerTest'/> + <test name='net.sf.briar.transport.SegmentedIncomingEncryptionLayerTest'/> + <test name='net.sf.briar.transport.SegmentedOutgoingEncryptionLayerTest'/> <test name='net.sf.briar.transport.XorErasureCodeTest'/> <test name='net.sf.briar.transport.XorErasureDecoderTest'/> <test name='net.sf.briar.transport.XorErasureEncoderTest'/> diff --git a/test/net/sf/briar/transport/ConnectionReaderImplTest.java b/test/net/sf/briar/transport/ConnectionReaderImplTest.java index feaddea5ce..3559501c4d 100644 --- a/test/net/sf/briar/transport/ConnectionReaderImplTest.java +++ b/test/net/sf/briar/transport/ConnectionReaderImplTest.java @@ -224,7 +224,8 @@ public class ConnectionReaderImplTest extends TransportTest { IncomingAuthenticationLayer authentication = new IncomingAuthenticationLayerImpl(correction, mac, macKey); IncomingReliabilityLayer reliability = - new NullIncomingReliabilityLayer(authentication); + new IncomingReliabilityLayerImpl(authentication, + new NullFrameWindow()); return new ConnectionReaderImpl(reliability, false); } } diff --git a/test/net/sf/briar/transport/FrameReadWriteTest.java b/test/net/sf/briar/transport/FrameReadWriteTest.java index 5ca1908782..98f50444db 100644 --- a/test/net/sf/briar/transport/FrameReadWriteTest.java +++ b/test/net/sf/briar/transport/FrameReadWriteTest.java @@ -103,7 +103,8 @@ public class FrameReadWriteTest extends BriarTestCase { IncomingAuthenticationLayer authenticationIn = new IncomingAuthenticationLayerImpl(correctionIn, mac, macKey); IncomingReliabilityLayer reliabilityIn = - new NullIncomingReliabilityLayer(authenticationIn); + new IncomingReliabilityLayerImpl(authenticationIn, + new NullFrameWindow()); ConnectionReader reader = new ConnectionReaderImpl(reliabilityIn, false); InputStream in1 = reader.getInputStream(); diff --git a/test/net/sf/briar/transport/IncomingSegmentedEncryptionLayerTest.java b/test/net/sf/briar/transport/SegmentedIncomingEncryptionLayerTest.java similarity index 96% rename from test/net/sf/briar/transport/IncomingSegmentedEncryptionLayerTest.java rename to test/net/sf/briar/transport/SegmentedIncomingEncryptionLayerTest.java index 13243605ed..8f8f5bc31f 100644 --- a/test/net/sf/briar/transport/IncomingSegmentedEncryptionLayerTest.java +++ b/test/net/sf/briar/transport/SegmentedIncomingEncryptionLayerTest.java @@ -22,12 +22,12 @@ import org.junit.Test; import com.google.inject.Guice; import com.google.inject.Injector; -public class IncomingSegmentedEncryptionLayerTest extends BriarTestCase { +public class SegmentedIncomingEncryptionLayerTest extends BriarTestCase { private final Cipher tagCipher, segCipher; private final ErasableKey tagKey, segKey; - public IncomingSegmentedEncryptionLayerTest() { + public SegmentedIncomingEncryptionLayerTest() { super(); Injector i = Guice.createInjector(new CryptoModule()); CryptoComponent crypto = i.getInstance(CryptoComponent.class); @@ -65,7 +65,7 @@ public class IncomingSegmentedEncryptionLayerTest extends BriarTestCase { SegmentSource in = new ByteArraySegmentSource(ciphertext1); // Use the encryption layer to decrypt the ciphertext IncomingEncryptionLayer decrypter = - new IncomingSegmentedEncryptionLayer(in, tagCipher, segCipher, + new SegmentedIncomingEncryptionLayer(in, tagCipher, segCipher, tagKey, segKey, false, buffered); // First segment Segment s = new SegmentImpl(); @@ -116,7 +116,7 @@ public class IncomingSegmentedEncryptionLayerTest extends BriarTestCase { SegmentSource in = new ByteArraySegmentSource(ciphertext1); // Use the encryption layer to decrypt the ciphertext IncomingEncryptionLayer decrypter = - new IncomingSegmentedEncryptionLayer(in, tagCipher, segCipher, + new SegmentedIncomingEncryptionLayer(in, tagCipher, segCipher, tagKey, segKey, true, buffered); // First segment Segment s = new SegmentImpl(); diff --git a/test/net/sf/briar/transport/OutgoingSegmentedEncryptionLayerTest.java b/test/net/sf/briar/transport/SegmentedOutgoingEncryptionLayerTest.java similarity index 95% rename from test/net/sf/briar/transport/OutgoingSegmentedEncryptionLayerTest.java rename to test/net/sf/briar/transport/SegmentedOutgoingEncryptionLayerTest.java index 00d467e92c..22bb21208a 100644 --- a/test/net/sf/briar/transport/OutgoingSegmentedEncryptionLayerTest.java +++ b/test/net/sf/briar/transport/SegmentedOutgoingEncryptionLayerTest.java @@ -22,14 +22,14 @@ import org.junit.Test; import com.google.inject.Guice; import com.google.inject.Injector; -public class OutgoingSegmentedEncryptionLayerTest extends BriarTestCase { +public class SegmentedOutgoingEncryptionLayerTest extends BriarTestCase { private static final int MAC_LENGTH = 32; private final Cipher tagCipher, segCipher; private final ErasableKey tagKey, segKey; - public OutgoingSegmentedEncryptionLayerTest() { + public SegmentedOutgoingEncryptionLayerTest() { super(); Injector i = Guice.createInjector(new CryptoModule()); CryptoComponent crypto = i.getInstance(CryptoComponent.class); @@ -65,7 +65,7 @@ public class OutgoingSegmentedEncryptionLayerTest extends BriarTestCase { // Use the encryption layer to encrypt the plaintext ByteArraySegmentSink sink = new ByteArraySegmentSink(); OutgoingEncryptionLayer encrypter = - new OutgoingSegmentedEncryptionLayer(sink, Long.MAX_VALUE, + new SegmentedOutgoingEncryptionLayer(sink, Long.MAX_VALUE, tagCipher, segCipher, tagKey, segKey, false); Segment s = new SegmentImpl(); System.arraycopy(plaintext, 0, s.getBuffer(), 0, plaintext.length); @@ -113,7 +113,7 @@ public class OutgoingSegmentedEncryptionLayerTest extends BriarTestCase { // Use the encryption layer to encrypt the plaintext SegmentSink sink = new ByteArraySegmentSink(); OutgoingEncryptionLayer encrypter = - new OutgoingSegmentedEncryptionLayer(sink, Long.MAX_VALUE, + new SegmentedOutgoingEncryptionLayer(sink, Long.MAX_VALUE, tagCipher, segCipher, tagKey, segKey, true); Segment s = new SegmentImpl(); System.arraycopy(plaintext, 0, s.getBuffer(), 0, plaintext.length); -- GitLab