Fix potential memory leaks

Author: aacuevas

Source/Plugins/BasicSpikeDisplay/SpikeDetector/SpikeDetector.cpp

@@ -62,7 +62,8 @@ SpikeDetector::SpikeDetector()
         electrodeCounter.add(0);
     }
 
-    spikeBuffer = new uint8_t[MAX_SPIKE_BUFFER_LEN]; // MAX_SPIKE_BUFFER_LEN defined in SpikeObject.h
+    //spikeBuffer = new uint8_t[MAX_SPIKE_BUFFER_LEN]; // MAX_SPIKE_BUFFER_LEN defined in SpikeObject.h
+	spikeBuffer.malloc(MAX_SPIKE_BUFFER_LEN);
 
 }
 
@@ -235,9 +236,9 @@ int SpikeDetector::getChannel(int index, int i)
     return *(electrodes[index]->channels+i);
 }
 
-Array<SimpleElectrode*> SpikeDetector::getElectrodes()
+void SpikeDetector::getElectrodes(Array<SimpleElectrode*>& electrodeArray)
 {
-    return electrodes;
+	electrodeArray.addArray(electrodes);
 }
 
 SimpleElectrode* SpikeDetector::setCurrentElectrodeIndex(int i)

Source/Plugins/BasicSpikeDisplay/SpikeDetector/SpikeDetector.h

@@ -136,7 +136,7 @@ class SpikeDetector : public GenericProcessor
     StringArray getElectrodeNames();
 
     /** Returns array of electrodes. */
-    Array<SimpleElectrode*> getElectrodes();
+	void getElectrodes(Array<SimpleElectrode*>& electrodeArray);
 
     /** Returns array of electrodes. */
     SimpleElectrode* getActiveElectrode();
@@ -176,10 +176,11 @@ class SpikeDetector : public GenericProcessor
     int currentChannelIndex;
     int currentIndex;
 
-    uint8_t* spikeBuffer;///[256];
+   // uint8_t* spikeBuffer;///[256];
+	HeapBlock<uint8_t> spikeBuffer;
     int64 timestamp;
 
-    Array<SimpleElectrode*> electrodes;
+    OwnedArray<SimpleElectrode> electrodes;
     int uniqueID;
 
     // void createSpikeEvent(int& peakIndex,

Source/Plugins/BasicSpikeDisplay/SpikeDetector/SpikeDetectorEditor.cpp

@@ -332,7 +332,8 @@ void SpikeDetectorEditor::buttonEvent(Button* button)
 
         SpikeDetector* processor = (SpikeDetector*) getProcessor();
 
-        Array<SimpleElectrode*> electrodes = processor->getElectrodes();
+		Array<SimpleElectrode*> electrodes;
+		processor->getElectrodes(electrodes);
 
         for (int i = 0; i < electrodes.size(); i++)
         {

Source/Processors/RecordNode/OriginalRecording.cpp

@@ -29,10 +29,14 @@ OriginalRecording::OriginalRecording() : separateFiles(false),
     recordingNumber(0), experimentNumber(0),  zeroBuffer(1, 50000),
     eventFile(nullptr), messageFile(nullptr), lastProcId(0)
 {
-    continuousDataIntegerBuffer = new int16[10000];
+    /*continuousDataIntegerBuffer = new int16[10000];
     continuousDataFloatBuffer = new float[10000];
 
-    recordMarker = new char[10];
+    recordMarker = new char[10];*/
+	continuousDataIntegerBuffer.malloc(10000);
+	continuousDataFloatBuffer.malloc(10000);
+	recordMarker.malloc(10);
+
     for (int i = 0; i < 9; i++)
     {
         recordMarker[i] = i;
@@ -53,9 +57,9 @@ OriginalRecording::~OriginalRecording()
     {
         if (spikeFileArray[i] != nullptr) fclose(spikeFileArray[i]);
     }
-    delete continuousDataFloatBuffer;
+ /*   delete continuousDataFloatBuffer;
     delete continuousDataIntegerBuffer;
-    delete recordMarker;
+    delete recordMarker;*/
 }
 
 String OriginalRecording::getEngineID()

Source/Processors/RecordNode/OriginalRecording.h

@@ -88,15 +88,18 @@ class OriginalRecording : public RecordEngine
     /** Holds data that has been converted from float to int16 before
         saving.
     */
-    int16* continuousDataIntegerBuffer;
+	HeapBlock<int16> continuousDataIntegerBuffer;
+    //int16* continuousDataIntegerBuffer;
 
     /** Holds data that has been converted from float to int16 before
         saving.
     */
-    float* continuousDataFloatBuffer;
+	HeapBlock<float> continuousDataFloatBuffer;
+    //float* continuousDataFloatBuffer;
 
     /** Used to indicate the end of each record */
-    char* recordMarker;
+	HeapBlock<char> recordMarker;
+    //char* recordMarker;
 
     AudioSampleBuffer zeroBuffer;