Skip to content

Commit 924c50b

Browse files
committed
Fix MinGW -Werror=array-bounds compiler error by using volatile indexes in LayersAndNeuronsContainerTest
1 parent d542d7d commit 924c50b

1 file changed

Lines changed: 7 additions & 5 deletions

File tree

tests/layers_and_neurons_container_tests.cpp

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -137,13 +137,15 @@ TEST(LayersAndNeuronsContainerTest, ValidationLogic)
137137
LayersAndNeuronsContainer container(topology);
138138

139139
// Layer out of bounds
140-
EXPECT_THROW(container.set(2, 0, 1.0), std::runtime_error);
141-
EXPECT_THROW((void)container.get_raw_ptr(2), std::runtime_error);
142-
EXPECT_THROW((void)container.get_span(2), std::runtime_error);
140+
volatile unsigned invalid_layer = 2;
141+
EXPECT_THROW(container.set(invalid_layer, 0, 1.0), std::runtime_error);
142+
EXPECT_THROW((void)container.get_raw_ptr(invalid_layer), std::runtime_error);
143+
EXPECT_THROW((void)container.get_span(invalid_layer), std::runtime_error);
143144

144145
// Neuron out of bounds
145-
EXPECT_THROW(container.set(0, 2, 1.0), std::runtime_error);
146-
EXPECT_THROW((void)container.get(0, 2), std::runtime_error);
146+
volatile unsigned invalid_neuron = 2;
147+
EXPECT_THROW(container.set(0, invalid_neuron, 1.0), std::runtime_error);
148+
EXPECT_THROW((void)container.get(0, invalid_neuron), std::runtime_error);
147149

148150
// Data size mismatch
149151
std::vector<double> too_much_data = { 1.0, 2.0, 3.0 };

0 commit comments

Comments
 (0)