seems like unintentional... +1 to remove

On Wed, May 8, 2013 at 8:16 PM, Shai Erera <[email protected]> wrote:
> 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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to