diff --git a/Code/Framework/AzCore/AzCore/Math/Random.h b/Code/Framework/AzCore/AzCore/Math/Random.h index c30bc4ddb6..8b2763df50 100644 --- a/Code/Framework/AzCore/AzCore/Math/Random.h +++ b/Code/Framework/AzCore/AzCore/Math/Random.h @@ -127,9 +127,9 @@ namespace AZ m_increments.fill(1); // By default increment by 1 between each number. } - //! Fills a provided container from begin to end with a Halton sequence - //! Entries are expected to be, or implicitely convert to, AZStd::array - template + //! Fills a provided container from begin to end with a Halton sequence. + //! Entries are expected to be, or implicitly converted to, AZStd::array. + template void FillHaltonSequence(Iterator begin, Iterator end) { AZStd::array indices = m_offsets; @@ -149,7 +149,7 @@ namespace AZ AZStd::generate(begin, end, f); } - //! Returns a Halton sequence in an array of N length + //! Returns a Halton sequence in an array of N length. template AZStd::array, N> GetHaltonSequence() { @@ -159,7 +159,7 @@ namespace AZ } //! Sets the offsets per dimension to start generating a sequence from. - //! By default, there is no offset (offset of 0 corresponds to starting at index 1) + //! By default, there is no offset (offset of 0 corresponds to starting at index 1). void SetOffsets(AZStd::array offsets) { m_offsets = offsets; diff --git a/Code/Framework/AzCore/Tests/Math/RandomTests.cpp b/Code/Framework/AzCore/Tests/Math/RandomTests.cpp index 95b92d21fe..7fe3acff54 100644 --- a/Code/Framework/AzCore/Tests/Math/RandomTests.cpp +++ b/Code/Framework/AzCore/Tests/Math/RandomTests.cpp @@ -103,7 +103,7 @@ namespace UnitTest AZStd::array ownedContainer; sequence.FillHaltonSequence(ownedContainer.begin(), ownedContainer.end()); - for (uint32_t i = 0; i < regularSequence.size(); ++i) + for (size_t i = 0; i < regularSequence.size(); ++i) { EXPECT_FLOAT_EQ(regularSequence[i][0], ownedContainer[i].x); EXPECT_FLOAT_EQ(regularSequence[i][1], ownedContainer[i].y);