Description Robert Konigsberg 2010-10-27 12:21:35 EDT

Build Identifier: M20100211-1343

I'm trying to do a large scale refactoring by inlining a method call. The inline is pretty simple in-and-of-itself:

/** @deprecated use {@code getInner().foo(); } */
public Bar foo() {
  return getInner().foo();
}

I told the inline operation to not remove foo() itself; I'm just hoping it will inline as many operations as possible.

The refactoring engine finds lots of uses, it even tells me about references it will disregard; references in class file, and "invalid references" (whatever those are.)

So the engine is smart enough to know that it can't catch all of the references to inline.

But when it comes across a read-only file, the entire inlining operation fails.

Reproducible: Always