On 17/10/2013 11:10 AM, Bradford Wetmore wrote:

http://cr.openjdk.java.net/~wetmore/8026762/webrev/

The JDK build broke because the JDK couldn't find the CORBA inbound files.

Needs to be quoted due to Windows separator being a semi-colon:

JAVAC:   is path separator
CYGWIN:  is statement/command separator

I also did two minor indention cleanups.  I know there are some inbound
CORBA cleanups, is this ok?  Please see the patch file for the complete
list, as "webrev -b" isn't working for whitespace changes.

I would leave off the indentation changes given what Erik was trying to achieve.

The actual bug fix is fine.

Thanks,
David

Brad

Reply via email to