Thanks for the updates, but libraries still have compilation issues:
1) trade.mq4 : typo switch: Lot -> lots , then later: lots -> templots
2) trade.mq4 : ATR not referenced
3) OA.mq4 & Lib.mq4 : event handling function not found ( here Fxpro mq4 now use metaeditor 915 which is tagged 5 not 4)
4) MayDay: ATR is declared/calculated here, so you need to choose if you still want it inside trade.mq4 (where it is not declared/calculated right now)
5) MayDay: dailyATR is used but its math is commented out (might be uncommented)
So before testing the EA, these libraries need first to be fixed to get testing results we could trust. 1 & 2 are easily fixed, but 3 clearly tell that the new coding (>600) style is expected.
Found that post that will help people : https://mqllock.com/common-compile-p...der-build-600/
ps: 3) above fixed by glancing at an other lib (stdlib.mq4) : all libs need to be declared as such; simply add : #property library
ps2: Mayday compile: declaration of 'lots' hides global declaration in file 'MayDay0.2.0.mq4' at line 114 Trade.mq4 44 44
So here also its the new rules applied with >600 version. (all the variables have to be inside the function itself, otherwise they disturb the global scope)
1) trade.mq4 : typo switch: Lot -> lots , then later: lots -> templots
2) trade.mq4 : ATR not referenced
3) OA.mq4 & Lib.mq4 : event handling function not found ( here Fxpro mq4 now use metaeditor 915 which is tagged 5 not 4)
4) MayDay: ATR is declared/calculated here, so you need to choose if you still want it inside trade.mq4 (where it is not declared/calculated right now)
5) MayDay: dailyATR is used but its math is commented out (might be uncommented)
So before testing the EA, these libraries need first to be fixed to get testing results we could trust. 1 & 2 are easily fixed, but 3 clearly tell that the new coding (>600) style is expected.
Found that post that will help people : https://mqllock.com/common-compile-p...der-build-600/
ps: 3) above fixed by glancing at an other lib (stdlib.mq4) : all libs need to be declared as such; simply add : #property library
ps2: Mayday compile: declaration of 'lots' hides global declaration in file 'MayDay0.2.0.mq4' at line 114 Trade.mq4 44 44
So here also its the new rules applied with >600 version. (all the variables have to be inside the function itself, otherwise they disturb the global scope)