[gcjx] Patch: FYI: end-of-constructor location

Tom Tromey tromey@redhat.com
Mon Mar 28 08:16:00 GMT 2005


I'm checking this in on the gcjx branch.

This fixes a little bug in a model_constructor constructor.  We were
calling get_location() on the wrong object, ultimately leading to
crashes when a constructor made this way was lowered to trees.

Tom

Index: ChangeLog
from  Tom Tromey  <tromey@redhat.com>
	* model/constructor.cc (model_constructor): Copy location of other
	constructor.

Index: model/constructor.cc
===================================================================
RCS file: /cvs/gcc/gcc/gcjx/model/Attic/constructor.cc,v
retrieving revision 1.1.2.2
diff -u -r1.1.2.2 constructor.cc
--- model/constructor.cc 12 Mar 2005 20:51:07 -0000 1.1.2.2
+++ model/constructor.cc 27 Mar 2005 04:59:45 -0000
@@ -38,7 +38,7 @@
 }
 
 model_constructor::model_constructor (model_constructor *other)
-  : model_method (get_location (), other->get_declaring_class ()),
+  : model_method (other->get_location (), other->get_declaring_class ()),
     other_this (NULL)
 {
 }



More information about the Java-patches mailing list