Skip to content

Commit 013cf80

Browse files
committed
Misc fixes.
1 parent 2f7afd3 commit 013cf80

5 files changed

Lines changed: 21 additions & 7 deletions

File tree

include/react/common/Containers.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
/***************************************/ REACT_IMPL_BEGIN /**************************************/
1717

1818
///////////////////////////////////////////////////////////////////////////////////////////////////
19-
/// BitField
19+
/// EnumFlags
2020
///////////////////////////////////////////////////////////////////////////////////////////////////
2121
template <typename T>
2222
class EnumFlags

include/react/common/Timing.h

Lines changed: 17 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -74,17 +74,31 @@ class ConditionalTimer<threshold,true>
7474
ScopedTimer(ConditionalTimer& parent) :
7575
parent_{ parent }
7676
{
77-
if (parent_.shouldMeasure_)
78-
QueryPerformanceCounter(&startTime_);
77+
if (!parent_.shouldMeasure_)
78+
return;
79+
80+
startMeasure();
7981
}
8082

81-
~ScopedTimer()
83+
// Note: virtual for performance reasons
84+
virtual ~ScopedTimer()
8285
{
8386
if (!parent_.shouldMeasure_)
8487
return;
8588

8689
parent_.shouldMeasure_ = false;
90+
91+
endMeasure();
92+
}
93+
94+
private:
95+
void startMeasure()
96+
{
97+
QueryPerformanceCounter(&startTime_);
98+
}
8799

100+
void endMeasure()
101+
{
88102
LARGE_INTEGER endTime, durationMS;
89103

90104
QueryPerformanceCounter(&endTime);
@@ -96,7 +110,6 @@ class ConditionalTimer<threshold,true>
96110
parent_.isThresholdExceeded_ = durationMS.QuadPart > threshold;
97111
}
98112

99-
private:
100113
ConditionalTimer& parent_;
101114

102115
LARGE_INTEGER startTime_;

include/react/common/Util.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88

99
#include "react/detail/Defs.h"
1010

11-
#include <ctime>
1211
#include <functional>
1312
#include <tuple>
1413
#include <type_traits>

include/react/detail/graph/SignalNodes.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ class SignalNode : public ReactiveNode<D,S,S>
3434
SignalNode() = default;
3535

3636
template <typename T>
37-
SignalNode(T&& value) :
37+
explicit SignalNode(T&& value) :
3838
ReactiveNode{ },
3939
value_{ std::forward<T>(value) }
4040
{}

src/benchmark/BenchmarkBase.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,11 @@
77
#pragma once
88

99
#include <cfloat>
10+
#include <ctime>
1011
#include <stdio.h>
1112
#include <iostream>
1213
#include <fstream>
14+
#include <random>
1315

1416
#include "react/common/Util.h"
1517

0 commit comments

Comments
 (0)