Bug 509738

Description Joël DRIGO 2016-12-28 05:43:19 EST

Hi,

An extra parenthesis in method invocation with lambdas as parameters isn't detected by the Java parser. The class compiles, and runs, doing not what it should do. 

The environment is:
Version: Neon.2 Release (4.6.2)
Build id: 20161208-0600

Java 8_u102 (with javac, I get a compilation error).
Windows 8.1


Here is a snippet to show the problem:
public class Snippet {

	public static void main(String[] args) {

		System.out.println("starting..."); 
		
		if ( false ) {
			System.out.println("this is executed, but it should not !?");
		}
		else {
			
			System.out.println("this is still executed...");
			exampleMethod(42, id1-> (p1, id2)-> otherMethod())); // The last extra parenthesis causes the parser to fail
			
		}

		System.out.println("...finishing"); 

	}
	
	
	public static interface I1 {
		I2 create(String id);
	}
	
	public static interface I2 {

		Class1 get(Class2 something, String somethingElse);
		
	}
	
	public static class Class1 {
		
	}

	public static class Class2 {
		
	}

	public static void exampleMethod(int param1, I1 param2) {
		System.out.println("example method is invoked");
	}
	
	public static Class1 otherMethod() {
		System.out.println("other method is invoked");
		return new Class1();
	}
	
}

I should not compile, but it do. The result of the run is wrong:
starting...
this is executed, but it should not !?
this is still executed...

The parser is deceived and completely delirious. If you try the following code (I replaced method name with whatever), it compiles and runs too, with same result as previous:

public class Snippet {

	public static void main(String[] args) {

		System.out.println("starting..."); 
		
		if ( false ) {
			System.out.println("this is executed, but it should not !?");
		}
		else {
			
			System.out.println("this is still executed...");
			something(42, id1-> (p1, id2)-> crazy())); // The last extra parenthesis causes the parser to fail
			
		}

		System.out.println("...finishing"); 

	}
	
	
	public static interface I1 {
		I2 create(String id);
	}
	
	public static interface I2 {

		Class1 get(Class2 something, String somethingElse);
		
	}
	
	public static class Class1 {
		
	}

	public static class Class2 {
		
	}

	public static void exampleMethod(int param1, I1 param2) {
		System.out.println("example method is invoked");
	}
	
	public static Class1 otherMethod() {
		System.out.println("other method is invoked");
		return new Class1();
	}
	
}