MockDirWrapper always syncs

[Shai Erera]

Hi

Look at this code from MDW.sync:

    if (true || LuceneTestCase.rarely(randomState) || delegate instanceof
NRTCachingDirectory) {
      // don't wear out our hardware so much in tests.
      delegate.sync(names);
    }

Is the 'if (true)' intentional or left there by mistake? The comment
afterwards suggests it should not be there, but I wanted to confirm before
I remove it.

Shai