Skip to content

Commit

Permalink
Fixes stack overflow exception in ReadSpectrogram
Browse files Browse the repository at this point in the history
The changes in a29aad1 introduced a self-calling version of the the ReadSpectrogram method when the overload was simplified.
This led to a stackoverflow and the difficult to debug variable unit test symptom we were seeing.

Work done for #196
  • Loading branch information
atruskie committed Mar 19, 2020
1 parent 25a77b2 commit 9b444ff
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions src/AudioAnalysisTools/Indices/IndexMatrices.cs
Original file line number Diff line number Diff line change
Expand Up @@ -420,11 +420,6 @@ public static Dictionary<string, double[]> AddDerivedIndices(Dictionary<string,
/// and the first column contains row/time names.
/// Note: no rotation of data is done!.
/// </summary>
public static double[,] ReadSpectrogram(FileInfo csvFile, out int binCount)
{
return ReadSpectrogram(csvFile, out binCount);
}

public static double[,] ReadSpectrogram(FileInfo csvFile, out int binCount, TwoDimensionalArray transform = TwoDimensionalArray.None)
{
double[,] matrix = Csv.ReadMatrixFromCsv<double>(csvFile, transform);
Expand Down

0 comments on commit 9b444ff

Please sign in to comment.