Skip to content

Commit

Permalink
remove no-argument constructor for RNGState, in favor of providing a …
Browse files Browse the repository at this point in the history
…default value for one of the one-argument constructors
  • Loading branch information
rileyjmurray committed Sep 28, 2024
1 parent a28b42d commit 42dcb2e
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions RandBLAS/base.hh
Original file line number Diff line number Diff line change
Expand Up @@ -101,9 +101,9 @@ struct RNGState {
/// this->key.incr(step).
typename RNG::key_type key;


/// Initialize the counter and key arrays to all zeros.
RNGState() : counter{{0}}, key(key_type{{}}) {}
/// Initialize the counter array to all zeros. Initialize the key array to have first
/// element equal to k and all other elements equal to zero.
RNGState(key_uint k = 0) : counter{{0}}, key{{k}} {}

// construct from a key
RNGState(key_type const &k) : counter{{0}}, key(k) {}
Expand All @@ -114,10 +114,6 @@ struct RNGState {
// move construct from an initial counter and key
RNGState(ctr_type &&c, key_type &&k) : counter(std::move(c)), key(std::move(k)) {}

/// Initialize the counter array to all zeros. Initialize the key array to have first
/// element equal to k and all other elements equal to zero.
RNGState(key_uint k) : counter{{0}}, key{{k}} {}

~RNGState() {};

/// A copy constructor.
Expand Down

0 comments on commit 42dcb2e

Please sign in to comment.