disregard

Author: rosasurfer

mql40/experts/ZigZag EA.mq4

@@ -1207,7 +1207,7 @@ bool SynchronizeStatus() {
    }
    SS.All();
 
-   if (open.ticket!=prevOpenTicket || ArrayRange(history, 0)!=prevHistorySize) {
+   if (open.ticket != prevOpenTicket || ArrayRange(history, 0) != prevHistorySize) {
       CalculateStats(true);
       return(SaveStatus());                                          // immediately save status if orders changed
    }

mql40/include/rsf/experts/status/file/SaveStatus.TradeHistory.mqh

@@ -13,18 +13,20 @@ bool SaveStatus.TradeHistory(string file, bool fileExists) {
    if (!fileExists) separator = CRLF;                        // an empty line separator
 
    double netProfit, netProfitP, sigProfitP;
+   int sizeHistory = ArrayRange(history, 0), sizePartials = ArrayRange(partialClose, 0);
 
-   string section = "Trade history";
-   int size = ArrayRange(history, 0);
-   for (int i=0; i < size; i++) {
-      WriteIniString(file, section, "full."+ i, HistoryRecordToStr(i, false));
+   string section = "Trade history", suffix = "";
+   for (int i=0; i < sizeHistory; i++) {
+      if (sizePartials && i == sizeHistory-1) {
+         suffix = separator;
+      }
+      WriteIniString(file, section, "full."+ i, HistoryRecordToStr(i, false) + suffix);
       netProfit  += history[i][H_NETPROFIT_M ];
       netProfitP += history[i][H_NETPROFIT_P ];
       sigProfitP += history[i][H_SIG_PROFIT_P];
    }
 
-   size = ArrayRange(partialClose, 0);
-   for (i=0; i < size; i++) {
+   for (i=0; i < sizePartials; i++) {
       WriteIniString(file, section, "part."+ i, HistoryRecordToStr(i, true));
    }
 

mql40/include/rsf/experts/trade/AddHistoryRecord.mqh

@@ -92,7 +92,7 @@ int AddHistoryRecord(int ticket, int fromTicket, int toTicket, int type, double
    else {
       // resolve the history[] index to insert at
       size = ArrayRange(history, 0);
-      for (i=size-1; i >= 0; i--) {                            // iterate from the end (in most use cases faster)
+      for (i=size-1; i >= 0; i--) {                            // iterate from the end (faster in most cases)
          if (ticket == history[i][H_TICKET]) return(_EMPTY(catch("AddHistoryRecord(2)  "+ instance.name +" cannot add record, ticket #"+ ticket +" already exists (history["+ i +"])", ERR_INVALID_PARAMETER)));
 
          if (openTime > history[i][H_OPENTIME]) {