Commit 013cf80
committed
File tree
- include/react
- common
- detail/graph
- src/benchmark
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -16,7 +16,7 @@ | |||
| 16 | 16 | /***************************************/ REACT_IMPL_BEGIN /**************************************/ | |
| 17 | 17 | ||
| 18 | 18 | /////////////////////////////////////////////////////////////////////////////////////////////////// | |
| 19 | - /// BitField | ||
| 19 | + /// EnumFlags | ||
| 20 | 20 | /////////////////////////////////////////////////////////////////////////////////////////////////// | |
| 21 | 21 | template <typename T> | |
| 22 | 22 | class EnumFlags | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -74,17 +74,31 @@ class ConditionalTimer<threshold,true> | |||
| 74 | 74 | ScopedTimer(ConditionalTimer& parent) : | |
| 75 | 75 | parent_{ parent } | |
| 76 | 76 | { | |
| 77 | - if (parent_.shouldMeasure_) | ||
| 78 | - QueryPerformanceCounter(&startTime_); | ||
| 77 | + if (!parent_.shouldMeasure_) | ||
| 78 | + return; | ||
| 79 | + | ||
| 80 | + startMeasure(); | ||
| 79 | 81 | } | |
| 80 | 82 | ||
| 81 | - ~ScopedTimer() | ||
| 83 | + // Note: virtual for performance reasons | ||
| 84 | + virtual ~ScopedTimer() | ||
| 82 | 85 | { | |
| 83 | 86 | if (!parent_.shouldMeasure_) | |
| 84 | 87 | return; | |
| 85 | 88 | ||
| 86 | 89 | parent_.shouldMeasure_ = false; | |
| 90 | + | ||
| 91 | + endMeasure(); | ||
| 92 | + } | ||
| 93 | + | ||
| 94 | + private: | ||
| 95 | + void startMeasure() | ||
| 96 | + { | ||
| 97 | + QueryPerformanceCounter(&startTime_); | ||
| 98 | + } | ||
| 87 | 99 | ||
| 100 | + void endMeasure() | ||
| 101 | + { | ||
| 88 | 102 | LARGE_INTEGER endTime, durationMS; | |
| 89 | 103 | ||
| 90 | 104 | QueryPerformanceCounter(&endTime); | |
@@ -96,7 +110,6 @@ class ConditionalTimer<threshold,true> | |||
| 96 | 110 | parent_.isThresholdExceeded_ = durationMS.QuadPart > threshold; | |
| 97 | 111 | } | |
| 98 | 112 | ||
| 99 | - private: | ||
| 100 | 113 | ConditionalTimer& parent_; | |
| 101 | 114 | ||
| 102 | 115 | LARGE_INTEGER startTime_; | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -8,7 +8,6 @@ | |||
| 8 | 8 | ||
| 9 | 9 | #include "react/detail/Defs.h" | |
| 10 | 10 | ||
| 11 | - #include <ctime> | ||
| 12 | 11 | #include <functional> | |
| 13 | 12 | #include <tuple> | |
| 14 | 13 | #include <type_traits> | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -34,7 +34,7 @@ class SignalNode : public ReactiveNode<D,S,S> | |||
| 34 | 34 | SignalNode() = default; | |
| 35 | 35 | ||
| 36 | 36 | template <typename T> | |
| 37 | - SignalNode(T&& value) : | ||
| 37 | + explicit SignalNode(T&& value) : | ||
| 38 | 38 | ReactiveNode{ }, | |
| 39 | 39 | value_{ std::forward<T>(value) } | |
| 40 | 40 | {} | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -7,9 +7,11 @@ | |||
| 7 | 7 | #pragma once | |
| 8 | 8 | ||
| 9 | 9 | #include <cfloat> | |
| 10 | + #include <ctime> | ||
| 10 | 11 | #include <stdio.h> | |
| 11 | 12 | #include <iostream> | |
| 12 | 13 | #include <fstream> | |
| 14 | + #include <random> | ||
| 13 | 15 | ||
| 14 | 16 | #include "react/common/Util.h" | |
| 15 | 17 | ||
0 commit comments