// Самый компактный способ получения коллекции
private IEnumerable<MyItem> GetActiveItems() => m_items.Where(it => it.IsActive);

// Использование
private void DoSomething()
{
    // Результат тот же самый, что и у вас. Возможен небольшой выигрыш в скорости выполнения
    // из-за оптимизированного метода ToList
    var activeItemsList = GetActiveItems.ToList();
    // То же самое, но выборка сохраняется в массив.
    var activeItemsArray = GetActiveItems.ToArray();
}

// Эквивалентный вариант. Более похожий на ваш.
private IEnumerable<MyItem> GetActiveItems2()
{
    foreach (var it in m_items)
    {
        if (it.IsActive)
        {
            yield return it;
        }
    }
}