Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: harmony-dev@incubator.apache.org
DomainKey-Status: good
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=n4/cMcVFsqWGHAnFNUiWYiIsTOuSrN8GKQofcvBxVZ9HJwvxZEWu+3/wZvpiazFOIT+sSM5RAS61OVDvsx6WAlWogXHKlLBXdjvhyxlkBRpdt7Y7nCQOUJXOX4b13QptQyo2lM7I31PzB4/I7VbyOutsdselb4jM2/d6GFB/u1g=
Message-ID: <e29119980610052355u5e17f7bficb36b563afa3e9a0@mail.gmail.com>
Date: Fri, 6 Oct 2006 13:55:30 +0700
From: "Evgueni Brevnov" <evgueni.brevnov@gmail.com>
To: harmony-dev@incubator.apache.org, geir@pobox.com
Subject: Re: [drlvm] too tired, need help
In-Reply-To: <4525F8C1.5050109@pobox.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <4525EB2E.6020301@pobox.com>
	 <4dd1f3f00610052257w7fad9e8eq86d5111cab7d5ee5@mail.gmail.com>
	 <4525F0BE.50507@pobox.com>
	 <ca4b22060610052308t9c4f1ebl75072018d37a244d@mail.gmail.com>
	 <4525F3C3.6010902@pobox.com> <4525F60C.3030608@pobox.com>
	 <e29119980610052327q5948f54g5eb9030202c09b76@mail.gmail.com>
	 <4525F8C1.5050109@pobox.com>

I can't provide you with the fresh patch right now. I want to restrore
cunit tests first.
If it doesn't introduce many changes then go ahead and commit. It
should be easy to merge then.

Evgueni

On 10/6/06, Geir Magnusson Jr. <geir@pobox.com> wrote:
> And it would only introduce a few, as far as I can tell, as only jni.cpp
> and vm_main.cpp actually include init.h, and therefore are the only
> files that depend on that global
>
> geir
>
> Evgueni Brevnov wrote:
> > Geir,
> >
> > Please don't do that until HARMONY-1582 integration. It can introduce
> > many conflicts.
> >
> > Tnanks in advance.
> > Evgueni
> >
> > On 10/6/06, Geir Magnusson Jr. <geir@pobox.com> wrote:
> >> I got it - thanks to Pavel.
> >>
> >> There's an extern in init.h, and it's declared in vm_main.cpp.
> >>
> >> Would anyone mind if I renamed it something less innocuous like
> >>
> >> global_env
> >> vm_global_env
> >>
> >> Something that gives the reader the hint that it's not a local var.
> >>
> >> geir
> >>
> >>
> >> Geir Magnusson Jr. wrote:
> >> > Thank you.
> >> >
> >> > How does the compiler know this when compiling jni.cpp?  is there an
> >> > extern defn somewhere?
> >> >
> >> > geir
> >> >
> >> > Pavel Rebriy wrote:
> >> >> File vm/vmcore/src/init/vm_main.cpp, line 68.
> >> >>
> >> >> Global_Env env(m, properties);
> >> >>
> >> >> On 06/10/06, Geir Magnusson Jr. <geir@pobox.com> wrote:
> >> >>>
> >> >>> I know the type.  The question is where is that variable "env"
> >> declared
> >> >>> and set?
> >> >>>
> >> >>> geir
> >> >>>
> >> >>>
> >> >>>
> >> >>> Weldon Washburn wrote:
> >> >>> > The first parameter to create_vm() is of type Global_Env.
> >> >>> >
> >> >>> > Global_Env is defined in environment.h
> >> >>> >
> >> >>> > vm_init() in vm_init.cpp initializes a bunch of Globla_Env's
> >> members.
> >> >>> >
> >> >>> >
> >> >>> >
> >> >>> >
> >> >>> > On 10/5/06, Geir Magnusson Jr. <geir@pobox.com> wrote:
> >> >>> >>
> >> >>> >> I'm trying to trace through the boot sequence chasing some boot
> >> >>> >> classpath property thing (luniglob sets it, and I can't figure
> >> how it
> >> >>> >> gets to us...), and I'm too tired, too dumb, or both to figure
> >> this
> >> >>> out.
> >> >>> >>
> >> >>> >> Launcher calls JNI_CreateJavaVM.  In our
> >> vmcore/src/jni/jni.cpp, we
> >> >>> >> define it, and it is  :
> >> >>> >>
> >> >>> >> VMEXPORT jint
> >> >>> >> JNICALL JNI_CreateJavaVM(JavaVM **p_vm, JNIEnv **p_env, void
> >> >>> *vm_args)
> >> >>> {
> >> >>> >>
> >> >>> >>     static int called = 0;
> >> >>> >>
> >> >>> >>     init_log_system();
> >> >>> >>     TRACE2("jni", "CreateJavaVM called");
> >> >>> >>     if (called) {
> >> >>> >>         ASSERT(0, "Not implemented");
> >> >>> >>         return JNI_ERR;
> >> >>> >>     } else {
> >> >>> >>         create_vm(&env, (JavaVMInitArgs *)vm_args);
> >> >>> >>         *p_env = &jni_env;
> >> >>> >>         *p_vm = jni_env.vm;
> >> >>> >>         return JNI_OK;
> >> >>> >>     }
> >> >>> >> }
> >> >>> >>
> >> >>> >> For the life of me, I can't figure out where "env" is defined
> >> or set.
> >> >>> >> create_vm() uses it...
> >> >>> >>
> >> >>> >> Can anyone give me a hint?  Eclipse's C++ plugin seems to be
> >> useless
> >> >>> >> here...
> >> >>> >>
> >> >>> >> geir
> >> >>> >>
> >> >>> >>
> >> ---------------------------------------------------------------------
> >> >>> >> Terms of use : http://incubator.apache.org/harmony/mailing.html
> >> >>> >> To unsubscribe, e-mail:
> >> harmony-dev-unsubscribe@incubator.apache.org
> >> >>> >> For additional commands, e-mail:
> >> >>> harmony-dev-help@incubator.apache.org
> >> >>> >>
> >> >>> >>
> >> >>> >
> >> >>> >
> >> >>>
> >> >>> ---------------------------------------------------------------------
> >> >>> Terms of use : http://incubator.apache.org/harmony/mailing.html
> >> >>> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> >> >>> For additional commands, e-mail:
> >> harmony-dev-help@incubator.apache.org
> >> >>>
> >> >>>
> >> >>
> >> >>
> >> >
> >> > ---------------------------------------------------------------------
> >> > Terms of use : http://incubator.apache.org/harmony/mailing.html
> >> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> >> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >> >
> >>
> >> ---------------------------------------------------------------------
> >> Terms of use : http://incubator.apache.org/harmony/mailing.html
> >> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> >> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >>
> >>
> >
> > ---------------------------------------------------------------------
> > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org