WIP. Should fix errors in MT4, MT5 & C++ for TaskRunner.

EA31337 · Feb 9, 2023 · 032e8e7 · 032e8e7
1 parent 720ec62
commit 032e8e7
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 6 deletions.
diff --git a/.github/workflows/compile-cpp.yml b/.github/workflows/compile-cpp.yml
@@ -43,6 +43,8 @@ jobs:
         with:
           compiler: gcc-latest
       - name: Compile ${{ matrix.file }} via emcc
+        if: always()
         run: emcc "${{ matrix.file }}"
       - name: Compile ${{ matrix.file }} via g++
+        if: always()
         run: g++ -c "${{ matrix.file }}"
diff --git a/Chart.struct.tf.h b/Chart.struct.tf.h
@@ -195,7 +195,12 @@ struct ChartTf {
   static unsigned int TfToSeconds(const ENUM_TIMEFRAMES _tf) {
     switch (_tf) {
       case PERIOD_CURRENT:
-        return PeriodSeconds(_tf);
+#ifdef __MQL__
+        return ::PeriodSeconds(_tf);
+#else
+        RUNTIME_ERROR("PeriodSeconds(PERIOD_CURRENT) is not implemented! Returning 0.");
+        return 0;
+#endif
       case PERIOD_M1:  // 1 minute.
         return 60;
       case PERIOD_M2:  // 2 minutes (non-standard).

diff --git a/Indicator/IndicatorData.h b/Indicator/IndicatorData.h
@@ -44,7 +44,6 @@ struct ExternInstantiateIndicatorBufferValueStorageDouble {
 #include "../Flags.h"
 #include "../Storage/IValueStorage.h"
 #include "../Storage/ItemsHistory.h"
-#include "../Storage/ValueStorage.indicator.h"
 #include "../SymbolInfo.struct.h"
 #include "Indicator.enum.h"
 #include "IndicatorBase.h"
@@ -1940,6 +1939,10 @@ int CopyBuffer(IndicatorData* _indi, int _mode, int _start, int _count, ValueSto
   return _num_copied;
 }
 
+// clang-format off
+#include "../Storage/ValueStorage.indicator.h"
+// clang-format on
+
 IValueStorage* ExternInstantiateIndicatorBufferValueStorageDouble::InstantiateIndicatorBufferValueStorageDouble(
     IndicatorData* _indi, int _mode) {
   return new IndicatorBufferValueStorage<double>(_indi, _mode);

diff --git a/Storage/ValueStorage.history.h b/Storage/ValueStorage.history.h
@@ -65,7 +65,7 @@ class HistoryValueStorage : public ValueStorage<C> {
   /**
    * Initializes storage with given value.
    */
-  virtual void Initialize(C _value) {
+  virtual void Initialize(C _value) override {
     Print("HistoryValueStorage does not implement Initialize()!");
     DebugBreak();
   }

diff --git a/Storage/ValueStorage.indicator.h b/Storage/ValueStorage.indicator.h
@@ -32,6 +32,8 @@
 
 // Forward declarations.
 class IndicatorData;
+template <typename C>
+class HistoryValueStorage;
 
 // Includes.
 #include "../Indicator/IndicatorData.h"
@@ -50,10 +52,13 @@ class IndicatorBufferValueStorage : public HistoryValueStorage<C> {
    * Constructor.
    */
   IndicatorBufferValueStorage(IndicatorData* _indi_candle, int _mode = 0, bool _is_series = false)
-      : mode(_mode), HistoryValueStorage(_indi_candle) {}
+      : HistoryValueStorage<C>(_indi_candle), mode(_mode) {}
 
   /**
    * Fetches value from a given shift. Takes into consideration as-series flag.
    */
-  C Fetch(int _rel_shift) override { return indi_candle REF_DEREF GetValue<C>(mode, RealShift(_rel_shift)); }
+  C Fetch(int _rel_shift) override {
+    IndicatorData* _indi = THIS_ATTR indi_candle.Ptr();
+    return _indi PTR_DEREF GetValue<C>(mode, THIS_ATTR RealShift(_rel_shift));
+  }
 };
diff --git a/Tick/Tick.struct.h b/Tick/Tick.struct.h
@@ -51,7 +51,7 @@ struct MqlTick {
   MqlTick() {}
 
   // Copy constructor.
-  MqlTick(){const MqlTick & r} {
+  MqlTick(const MqlTick &r) {
     time = r.time;
     ask = r.ask;
     bid = r.bid;