[Rust] Take slices by reference. (aeron-io#979)

Author: gierlachg

rust/tests/baseline_test.rs

@@ -176,8 +176,8 @@ fn encode_car_from_scratch() -> SbeResult<(usize, Vec<u8>)> {
  car.model_year(2013);
  car.available(BooleanType::T);
  car.code(Model::A);
- car.some_numbers([1, 2, 3, 4]);
- car.vehicle_code([97, 98, 99, 100, 101, 102]); // abcdef
+ car.some_numbers(&[1, 2, 3, 4]);
+ car.vehicle_code(&[97, 98, 99, 100, 101, 102]); // abcdef
 
  extras.set_cruise_control(true);
  extras.set_sports_pack(true);
@@ -187,7 +187,7 @@ fn encode_car_from_scratch() -> SbeResult<(usize, Vec<u8>)> {
  let mut engine = car.engine_encoder();
  engine.capacity(2000);
  engine.num_cylinders(4);
- engine.manufacturer_code([49, 50, 51]); // 123
+ engine.manufacturer_code(&[49, 50, 51]); // 123
  engine.efficiency(35);
  engine.booster_enabled(BooleanType::T);
  let mut booster = engine.booster_encoder();

rust/tests/big_endian_test.rs

@@ -170,8 +170,8 @@ fn encode_car_from_scratch() -> SbeResult<(usize, Vec<u8>)> {
  car.model_year(2013);
  car.available(BooleanType::T);
  car.code(Model::A);
- car.some_numbers([0, 1, 2, 3, 4]);
- car.vehicle_code(*b"abcdef");
+ car.some_numbers(&[0, 1, 2, 3, 4]);
+ car.vehicle_code(b"abcdef");
 
  extras.set_cruise_control(true);
  extras.set_sports_pack(true);
@@ -181,7 +181,7 @@ fn encode_car_from_scratch() -> SbeResult<(usize, Vec<u8>)> {
  let mut engine = car.engine_encoder();
  engine.capacity(2000);
  engine.num_cylinders(4);
- engine.manufacturer_code(*b"123");
+ engine.manufacturer_code(b"123");
  engine.efficiency(35);
  engine.booster_enabled(BooleanType::T);
  let mut booster = engine.booster_encoder();

rust/tests/extension_test.rs

@@ -177,8 +177,8 @@ fn encode_car_from_scratch() -> SbeResult<(usize, Vec<u8>)> {
  car.model_year(2013);
  car.available(BooleanType::T);
  car.code(Model::A);
- car.some_numbers([0, 1, 2, 3]);
- car.vehicle_code([97, 98, 99, 100, 101, 102]); // abcdef
+ car.some_numbers(&[0, 1, 2, 3]);
+ car.vehicle_code(&[97, 98, 99, 100, 101, 102]); // abcdef
 
  extras.set_cruise_control(true);
  extras.set_sports_pack(true);
@@ -188,7 +188,7 @@ fn encode_car_from_scratch() -> SbeResult<(usize, Vec<u8>)> {
  let mut engine = car.engine_encoder();
  engine.capacity(2000);
  engine.num_cylinders(4);
- engine.manufacturer_code([49, 50, 51]); // 123
+ engine.manufacturer_code(&[49, 50, 51]); // 123
  engine.efficiency(35);
  engine.booster_enabled(BooleanType::T);
  let mut booster = engine.booster_encoder();

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

@@ -56,7 +56,7 @@ void generate() throws IOException
  {
  indent(libRs, 0, "#![forbid(unsafe_code)]\n");
  indent(libRs, 0, "#![allow(clippy::upper_case_acronyms)]\n");
- indent(libRs, 0, "#![allow(non_camel_case_types)]\n");
+ indent(libRs, 0, "#![allow(non_camel_case_types)]\n\n");
  indent(libRs, 0, "use ::core::{convert::TryInto};\n\n");
 
  final ArrayList<String> modules = new ArrayList<>();
@@ -225,8 +225,8 @@ static void generateWriteBuf(final Writer writer, final ByteOrder byteOrder) thr
 
  indent(writer, 1, "#[inline]\n");
  indent(writer, 1,
- "pub fn put_bytes_at<const COUNT: usize>(&mut self, index: usize, bytes: [u8; COUNT]) -> usize {\n");
- indent(writer, 2, "self.data[index..index + COUNT].copy_from_slice(&bytes);\n");
+ "pub fn put_bytes_at<const COUNT: usize>(&mut self, index: usize, bytes: &[u8; COUNT]) -> usize {\n");
+ indent(writer, 2, "self.data[index..index + COUNT].copy_from_slice(bytes);\n");
  indent(writer, 2, "COUNT\n");
  indent(writer, 1, "}\n\n");
 
@@ -245,7 +245,7 @@ static void generateWriteBuf(final Writer writer, final ByteOrder byteOrder) thr
  // put_<primitive>_at
  indent(writer, 1, "#[inline]\n");
  indent(writer, 1, "pub fn put_%1$s_at(&mut self, index: usize, value: %1$s) {\n", primitiveType);
- indent(writer, 2, "self.put_bytes_at(index, %s::to_%s_bytes(value));\n", primitiveType, endianness);
+ indent(writer, 2, "self.put_bytes_at(index, &%s::to_%s_bytes(value));\n", primitiveType, endianness);
  indent(writer, 1, "}\n\n");
  }
 

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

@@ -379,7 +379,7 @@ private static void generatePrimitiveEncoder(
  indent(sb, level, "/// - encodedLength: %d\n", typeToken.encodedLength());
  indent(sb, level, "/// - version: %d\n", typeToken.version());
  indent(sb, level, "#[inline]\n");
- indent(sb, level, "pub fn %s(&mut self, value: [%s; %d]) {\n",
+ indent(sb, level, "pub fn %s(&mut self, value: &[%s; %d]) {\n",
  formatFunctionName(name),
  rustPrimitiveType,
  arrayLength);