RemoveWarmupPeriods has extra enumeration with ToList()

[DaveSkender]

Problem: extra list enumeration (performance)

• consider SkipWhile<T>(IEnumerable<T>, Func<T,Boolean>)
• or other non-LINQ, non-enumerating approaches

Other considerations

• also for Prune() and related utilities
• v3 approach coordination

// current v2 approach
public static IEnumerable<BollingerBandsResult> RemoveWarmupPeriods(
    this IEnumerable<BollingerBandsResult> results)
{
    int removePeriods = results
        .ToList()
        .FindIndex(x => x.Width != null);

    return results.Remove(removePeriods);
}

Discussed in #1310

^{Originally posted by **martonb** January 5, 2025} Just noticed, RemoveWarmupPeriods for Bollinger Bands uses a ToList, which looks unnecessary, to me... Any reason for it or just mistake?