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 ??
    }
}

并且以为他可以减少行数。

我会很担心是否会这样...

这是类似的问题，并回答：

锁可确保互斥-最多只能同时有一个线程持有该锁。该锁由特定的对象实例标识。您正在创建一个每次都要锁定的新对象，并且您没有任何方法可以通知任何其他线程锁定该完全相同的对象实例。因此，您的锁定是无用的。

这可能没用。但是，这有可能会造成内存障碍。不确定c＃是否执行锁省略或是否保留锁的排序语义。