fix: check the length of array properly

Signed-off-by: Yuuki Takano <yuuki.takano@tier4.jp>

src/msg.rs

@@ -99,7 +99,7 @@ macro_rules! def_sequence {
 
  impl<const N: usize> $ty<N> {
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -383,7 +383,7 @@ pub struct RosStringSeq<const STRLEN: usize, const SEQLEN: usize>(
 
 impl<const STRLEN: usize, const SEQLEN: usize> RosStringSeq<STRLEN, SEQLEN> {
  pub fn new(size: usize) -> Option<Self> {
- if SEQLEN != 0 && size >= SEQLEN {
+ if SEQLEN != 0 && size > SEQLEN {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/action_msgs/msg/goal_info.rs

@@ -66,7 +66,7 @@ impl<const N: usize> GoalInfoSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/action_msgs/msg/goal_status.rs

@@ -79,7 +79,7 @@ impl<const N: usize> GoalStatusSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/action_msgs/msg/goal_status_array.rs

@@ -71,7 +71,7 @@ impl<const N: usize> GoalStatusArraySeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/action_msgs/srv/cancel_goal.rs

@@ -86,7 +86,7 @@ impl<const N: usize> CancelGoalRequestSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -203,7 +203,7 @@ impl<const N: usize> CancelGoalResponseSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/composition_interfaces/srv/list_nodes.rs

@@ -86,7 +86,7 @@ impl<const N: usize> ListNodesRequestSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -204,7 +204,7 @@ impl<const N: usize> ListNodesResponseSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/composition_interfaces/srv/load_node.rs

@@ -95,7 +95,7 @@ impl<const N: usize> LoadNodeRequestSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -213,7 +213,7 @@ impl<const N: usize> LoadNodeResponseSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/composition_interfaces/srv/unload_node.rs

@@ -87,7 +87,7 @@ impl<const N: usize> UnloadNodeRequestSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -206,7 +206,7 @@ impl<const N: usize> UnloadNodeResponseSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/lifecycle_msgs/msg/state.rs

@@ -77,7 +77,7 @@ impl<const N: usize> StateSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/lifecycle_msgs/msg/transition.rs

@@ -102,7 +102,7 @@ impl<const N: usize> TransitionSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/lifecycle_msgs/msg/transition_description.rs

@@ -75,7 +75,7 @@ impl<const N: usize> TransitionDescriptionSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/lifecycle_msgs/msg/transition_event.rs

@@ -74,7 +74,7 @@ impl<const N: usize> TransitionEventSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/lifecycle_msgs/srv/change_state.rs

@@ -83,7 +83,7 @@ impl<const N: usize> ChangeStateRequestSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -201,7 +201,7 @@ impl<const N: usize> ChangeStateResponseSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/lifecycle_msgs/srv/get_available_states.rs

@@ -89,7 +89,7 @@ impl<const N: usize> GetAvailableStatesRequestSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -209,7 +209,7 @@ impl<const N: usize> GetAvailableStatesResponseSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/lifecycle_msgs/srv/get_available_transitions.rs

@@ -93,7 +93,7 @@ impl<const N: usize> GetAvailableTransitionsRequestSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -213,7 +213,7 @@ impl<const N: usize> GetAvailableTransitionsResponseSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/lifecycle_msgs/srv/get_state.rs

@@ -81,7 +81,7 @@ impl<const N: usize> GetStateRequestSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -198,7 +198,7 @@ impl<const N: usize> GetStateResponseSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/msg/floating_point_range.rs

@@ -73,7 +73,7 @@ impl<const N: usize> FloatingPointRangeSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/msg/integer_range.rs

@@ -73,7 +73,7 @@ impl<const N: usize> IntegerRangeSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/msg/list_parameters_result.rs

@@ -74,7 +74,7 @@ impl<const N: usize> ListParametersResultSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/msg/log.rs

@@ -76,7 +76,7 @@ impl<const N: usize> LogSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/msg/parameter.rs

@@ -72,7 +72,7 @@ impl<const N: usize> ParameterSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/msg/parameter_descriptor.rs

@@ -80,7 +80,7 @@ impl<const N: usize> ParameterDescriptorSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/msg/parameter_event.rs

@@ -75,7 +75,7 @@ impl<const N: usize> ParameterEventSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/msg/parameter_event_descriptors.rs

@@ -77,7 +77,7 @@ impl<const N: usize> ParameterEventDescriptorsSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/msg/parameter_type.rs

@@ -81,7 +81,7 @@ impl<const N: usize> ParameterTypeSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/msg/parameter_value.rs

@@ -80,7 +80,7 @@ impl<const N: usize> ParameterValueSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/msg/set_parameters_result.rs

@@ -74,7 +74,7 @@ impl<const N: usize> SetParametersResultSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/srv/describe_parameters.rs

@@ -89,7 +89,7 @@ impl<const N: usize> DescribeParametersRequestSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -209,7 +209,7 @@ impl<const N: usize> DescribeParametersResponseSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/srv/get_parameter_types.rs

@@ -89,7 +89,7 @@ impl<const N: usize> GetParameterTypesRequestSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -208,7 +208,7 @@ impl<const N: usize> GetParameterTypesResponseSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/srv/get_parameters.rs

@@ -85,7 +85,7 @@ impl<const N: usize> GetParametersRequestSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -203,7 +203,7 @@ impl<const N: usize> GetParametersResponseSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }

src/msg/humble/interfaces/rcl_interfaces/srv/list_parameters.rs

@@ -88,7 +88,7 @@ impl<const N: usize> ListParametersRequestSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }
@@ -206,7 +206,7 @@ impl<const N: usize> ListParametersResponseSeq<N> {
  /// `N` represents the maximum number of elements.
  /// If `N` is `0`, the sequence is unlimited.
  pub fn new(size: usize) -> Option<Self> {
- if N != 0 && size >= N {
+ if N != 0 && size > N {
  // the size exceeds in the maximum number
  return None;
  }