Attachment 194026 Details for Bug 343713 – previous patch with copyright updated

[patch] previous patch with copyright updated

Bug_343713_v3.patch (text/plain), 4.31 KB, created by Stephan Herrmann

on 2011-04-25 18:48:35 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Stephan Herrmann

Created: 2011-04-25 18:48:35 EDT

Size: 4.31 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.core
>Index: compiler/org/eclipse/jdt/internal/compiler/ast/ConstructorDeclaration.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/compiler/org/eclipse/jdt/internal/compiler/ast/ConstructorDeclaration.java,v
>retrieving revision 1.114
>diff -u -r1.114 ConstructorDeclaration.java
>--- compiler/org/eclipse/jdt/internal/compiler/ast/ConstructorDeclaration.java	5 Mar 2011 17:18:43 -0000	1.114
>+++ compiler/org/eclipse/jdt/internal/compiler/ast/ConstructorDeclaration.java	25 Apr 2011 22:44:33 -0000
>@@ -7,6 +7,7 @@
>  *
>  * Contributors:
>  *     IBM Corporation - initial API and implementation
>+ *     Stephan Herrmann - Contribution for Bug 343713 - [compiler] bogus line number in constructor of inner class in 1.5 compliance
>  *******************************************************************************/
> package org.eclipse.jdt.internal.compiler.ast;
> 
>@@ -335,6 +336,7 @@
> 
> 		if (needFieldInitializations && preInitSyntheticFields){
> 			generateSyntheticFieldInitializationsIfNecessary(this.scope, codeStream, declaringClass);
>+			codeStream.recordPositionsFrom(0, this.bodyStart);
> 		}
> 		// generate constructor call
> 		if (this.constructorCall != null) {
>#P org.eclipse.jdt.core.tests.compiler
>Index: src/org/eclipse/jdt/core/tests/compiler/regression/InnerEmulationTest_1_5.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/compiler/regression/InnerEmulationTest_1_5.java,v
>retrieving revision 1.1
>diff -u -r1.1 InnerEmulationTest_1_5.java
>--- src/org/eclipse/jdt/core/tests/compiler/regression/InnerEmulationTest_1_5.java	12 May 2009 20:49:53 -0000	1.1
>+++ src/org/eclipse/jdt/core/tests/compiler/regression/InnerEmulationTest_1_5.java	25 Apr 2011 22:44:35 -0000
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2009 IBM Corporation and others.
>+ * Copyright (c) 2009, 2011 IBM Corporation and others.
>  * All rights reserved. This program and the accompanying materials
>  * are made available under the terms of the Eclipse Public License v1.0
>  * which accompanies this distribution, and is available at
>@@ -7,6 +7,7 @@
>  *
>  * Contributors:
>  *     IBM Corporation - initial API and implementation
>+ *     Stephan Herrmann - Contribution for Bug 343713 - [compiler] bogus line number in constructor of inner class in 1.5 compliance
>  *******************************************************************************/
> package org.eclipse.jdt.core.tests.compiler.regression;
> 
>@@ -286,6 +287,41 @@
> 		"     inner name: #24 D, accessflags: 8 static]\n";
> 	checkDisassembledClassFile(OUTPUT_DIR + File.separator + "p" + File.separator + "X.class", "X", expectedOutput);
> }
>+// https://bugs.eclipse.org/bugs/show_bug.cgi?id=343713
>+// [compiler] bogus line number in constructor of inner class in 1.5 compliance
>+public void test14() throws Exception {
>+	runConformTest(new String[] {
>+		"LineNumberBug.java",
>+		"public class LineNumberBug {\n" + 
>+		"    class Inner {\n" + 
>+		"		public Inner() {\n" + 
>+		"			System.out.println(\"Inner()\");\n" + 
>+		"		}\n" + 
>+		"    }\n" + 
>+		"	public static void main(String[] args) {\n" + 
>+		"		new LineNumberBug().new Inner();\n" + 
>+		"	}\n" + 
>+		"}\n"
>+	});
>+	String expectedOutput =
>+		"  // Method descriptor #8 (LLineNumberBug;)V\n" + 
>+		"  // Stack: 2, Locals: 2\n" + 
>+		"  public LineNumberBug$Inner(LineNumberBug arg0);\n" + 
>+		"     0  aload_0 [this]\n" + 
>+		"     1  aload_1 [arg0]\n" + 
>+		"     2  putfield LineNumberBug$Inner.this$0 : LineNumberBug [10]\n" + 
>+		"     5  aload_0 [this]\n" + 
>+		"     6  invokespecial java.lang.Object() [12]\n" + 
>+		"     9  getstatic java.lang.System.out : java.io.PrintStream [15]\n" + 
>+		"    12  ldc <String \"Inner()\"> [21]\n" + 
>+		"    14  invokevirtual java.io.PrintStream.println(java.lang.String) : void [23]\n" + 
>+		"    17  return\n" + 
>+		"      Line numbers:\n" + 
>+		"        [pc: 0, line: 3]\n" + 
>+		"        [pc: 9, line: 4]\n" +
>+		"        [pc: 17, line: 5]\n";
>+	checkDisassembledClassFile(OUTPUT_DIR + File.separator + "LineNumberBug$Inner.class", "LineNumberBug$Inner", expectedOutput);	
>+}
> public static Class testClass() {
> 	return InnerEmulationTest_1_5.class;
> }

Actions: View | Diff

Attachments on bug 343713: 193971 | 193972 | 194026