NHibernate 2.1 short codereview

Um erst einmal eins vorweg zu sagen, ich mag NHibernate wirklich sehr. Ich will dieses Tool nicht schlecht machen oder so, auch wenn ich in letzter Zeit viel über Bugs oder komische Codestellen berichte. Es ist ein sehr gutes Tool und wenn mir die Qualität nicht passt habe ich selbst die Möglichkeit etwas dran zu ändern.

Da ich in letzter Zeit vermehrt in den Sourcecode von NHibernate reingeschaut habe, sind doch ein paar Stellen aufgefallen die ich einfach grottig fand. Eine von diesen werde ich kurz erläutern.

In der Klasse Configuration gibt es die Methode BuildSessionFactory(). Diese sieht momentan so aus:

public ISessionFactory BuildSessionFactory()
{

    ConfigureProxyFactoryFactory();
    SecondPassCompile();
    Validate();
    Environment.VerifyProperties(properties);
    Settings settings = BuildSettings();

    // Ok, don't need schemas anymore, so free them
    Schemas = null;

    return new SessionFactoryImpl(this, mapping, settings, GetInitializedEventListeners());
}

 Die rot markierte Zeile ist das Interessante an der Sache. Wenn ich so in den Qullcode schaue, würde ich ja denken hier werden Properties überprüft. Wenn wir uns nun mal die Methode anschauen stellen wir fest ...

/// <summary>
/// Issue warnings to user when any obsolete property names are used.
/// </summary>
/// <param name="props"></param>
/// <returns></returns>
public static void VerifyProperties(IDictionary<string, string> props) {}

sie macht garnichts!
Sehr cool finde ich den Kommentar.