Consider removing DirectiveCollection #1912

    // From guideline prose:
    // "When implementing such a custom collection, follow the API pattern
    // established by Collection<T> and ReadOnlyCollection<T> as
    // closely as possible. That is, implement the same members explicitly,
    // name the parameters in the same way that these two collections name
    // them, and so on. In other words, make your custom collection different
    // from these two collections only when you have a very good reason to
    // do so."
    public class DirectiveCollection : IEnumerable<KeyValuePair<string,IEnumerable<string>>>, IEnumerable 
    {
        public DirectiveCollection();
        public bool Contains(string name);
        public IEnumerator<KeyValuePair<string,IEnumerable<string>>> GetEnumerator();
        public bool TryGetValues(string name, ref IReadOnlyList<string> values);
    }