lock（new object（））—カーゴカルトまたはクレイジーな「言語の特殊なケース」？

コンサルタントによって作成されたコードを確認しています。すでに数十の危険信号が表示されていますが、次のスニペットに頭を悩ませることはできません。

private void foo()
{
    if (InvokeRequired)
    {
        lock (new object())
        {
            if (m_bar!= null)
                Invoke(new fooDelegate(foo), new object[] { });
        }
    }
    else
    {
        if(OnBazChanged != null)
            OnBazChanged();
    }
}

ここでlock（new object（））は何をしていますか？常に別のオブジェクトをロックしているため、何の効果もありませんが、この種のロックは、コピーして貼り付けられていない部分であっても、コード全体で持続します。これは、私が知らないものにコンパイルされたC＃言語の特殊なケースですか、それともプログラマーは、たまたま機能していたカーゴカルトを採用しただけですか？

これを見た人がいたとしても、私は驚かないでしょう。

private readonly object lockObj = new object();

private void MyMethod()
{
    lock(lockObj)
    {
        // do amazing stuff, so amazing it can only run once at a time
        // e.g. comands on the Mars Rover, or programs on iOS pre 4 / 5 ??
    }
}

そして彼は行数を減らすことができると思いました。

もしそうなら、私は非常に心配するでしょう...

ここで同様の質問、およびその答えは次のとおりです。

ロックは相互排除を保証します-同時にロックを保持できるスレッドは1つだけです。ロックは特定のオブジェクトインスタンスで識別されます。毎回ロックする新しいオブジェクトを作成していて、まったく同じオブジェクトインスタンスをロックするように他のスレッドに通知する方法がありません。したがって、ロックは役に立ちません。

おそらく役に立たないでしょう。しかし、メモリバリアを作成する可能性はほとんどありません。c＃がロックの省略を行うのか、それともロックの順序付けのセマンティクスを保持するのかはわかりません。