Skip to content

Commit 67a5cf1

Browse files
committed
[Java] Tidy up after merge of PR aeron-io#877.
1 parent 680e28f commit 67a5cf1

File tree

6 files changed

+17
-22
lines changed

6 files changed

+17
-22
lines changed

sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/rust/LibRsDef.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -60,10 +60,10 @@ void generate() throws IOException
6060
final ArrayList<String> modules = new ArrayList<>();
6161
Files.walk(outputManager.getSrcDirPath())
6262
.filter(Files::isRegularFile)
63-
.map(path -> path.getFileName().toString())
64-
.filter(fileName -> fileName.endsWith(".rs"))
65-
.filter(fileName -> !fileName.equals("lib.rs"))
66-
.map(fileName -> fileName.substring(0, fileName.length() - 3))
63+
.map((path) -> path.getFileName().toString())
64+
.filter((fileName) -> fileName.endsWith(".rs"))
65+
.filter((fileName) -> !fileName.equals("lib.rs"))
66+
.map((fileName) -> fileName.substring(0, fileName.length() - 3))
6767
.forEach(modules::add);
6868

6969
// add modules

sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/rust/MessageCoderDef.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,6 @@ void appendTo(final Appendable dest) throws IOException
119119
dest.append(sb);
120120
}
121121

122-
@Override
123122
public SubGroup addSubGroup(final String name, final int level, final Token groupToken)
124123
{
125124
final SubGroup subGroup = new SubGroup(name, level, groupToken);

sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/rust/RustGenerator.java

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -33,10 +33,11 @@
3333
import java.util.Set;
3434

3535
import static java.lang.String.format;
36-
import static uk.co.real_logic.sbe.generation.rust.RustGenerator.CodecType.Decoder;
37-
import static uk.co.real_logic.sbe.generation.rust.RustGenerator.CodecType.Encoder;
3836
import static uk.co.real_logic.sbe.generation.rust.RustUtil.*;
39-
import static uk.co.real_logic.sbe.ir.GenerationUtil.*;
37+
import static uk.co.real_logic.sbe.ir.GenerationUtil.collectFields;
38+
import static uk.co.real_logic.sbe.ir.GenerationUtil.collectGroups;
39+
import static uk.co.real_logic.sbe.ir.GenerationUtil.collectVarData;
40+
import static uk.co.real_logic.sbe.ir.GenerationUtil.getMessageBody;
4041
import static uk.co.real_logic.sbe.ir.Signal.BEGIN_ENUM;
4142
import static uk.co.real_logic.sbe.ir.Signal.BEGIN_SET;
4243

@@ -53,15 +54,15 @@ enum CodecType
5354
{
5455
Decoder
5556
{
56-
@Override String bufType()
57+
String bufType()
5758
{
5859
return READ_BUF_TYPE;
5960
}
6061
},
6162

6263
Encoder
6364
{
64-
@Override String bufType()
65+
String bufType()
6566
{
6667
return WRITE_BUF_TYPE;
6768
}
@@ -81,8 +82,8 @@ interface ParentDef
8182
/**
8283
* Create a new Rust language {@link CodeGenerator}.
8384
*
84-
* @param ir for the messages and types.
85-
* @param outputManager for generating the codecs to.
85+
* @param ir for the messages and types.
86+
* @param outputManager for generating the codecs to.
8687
*/
8788
public RustGenerator(
8889
final Ir ir,
@@ -227,9 +228,9 @@ static void generateEncoderFields(
227228
break;
228229
}
229230
}
230-
catch (final IOException e)
231+
catch (final IOException ex)
231232
{
232-
throw new RuntimeException(e);
233+
throw new RuntimeException(ex);
233234
}
234235
});
235236
}
@@ -1465,6 +1466,4 @@ private static void appendConstAccessor(
14651466
indent(writer, level + 1, rustExpression + "\n");
14661467
indent(writer, level, "}\n\n");
14671468
}
1468-
1469-
14701469
}

sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/rust/RustOutputManager.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ static File createDir(final String dirName)
4444
{
4545
throw new IllegalStateException("Unable to create directory: " + dirName);
4646
}
47+
4748
return dir;
4849
}
4950

@@ -77,7 +78,6 @@ public RustOutputManager(final String baseDirName, final String packageName)
7778
* @return a {@link java.io.Writer} to which the source code should be written.
7879
* @throws IOException if an issue occurs when creating the file.
7980
*/
80-
@Override
8181
public Writer createOutput(final String name) throws IOException
8282
{
8383
final String fileName = toLowerSnakeCase(name) + ".rs";
@@ -102,5 +102,4 @@ Path getSrcDirPath()
102102
{
103103
return srcDir.toPath();
104104
}
105-
106105
}

sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/rust/RustUtil.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,12 @@
3232
import static uk.co.real_logic.sbe.generation.rust.RustGenerator.CodecType.Encoder;
3333

3434
/**
35-
*
35+
* Utility method for Rust codec generation.
3636
*/
3737
public class RustUtil
3838
{
3939
static final String INDENT = " ";
40-
public static final Map<PrimitiveType, String> TYPE_NAME_BY_PRIMITIVE_TYPE_MAP =
41-
new EnumMap<>(PrimitiveType.class);
40+
static final Map<PrimitiveType, String> TYPE_NAME_BY_PRIMITIVE_TYPE_MAP = new EnumMap<>(PrimitiveType.class);
4241

4342
static
4443
{

sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/rust/SubGroup.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@ class SubGroup implements RustGenerator.ParentDef
4141
this.groupToken = groupToken;
4242
}
4343

44-
@Override
4544
public SubGroup addSubGroup(final String name, final int level, final Token groupToken)
4645
{
4746
final SubGroup subGroup = new SubGroup(name, level, groupToken);

0 commit comments

Comments
 (0)