You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@harmony.apache.org by "Fedotov, Alexei A" <al...@intel.com> on 2006/10/05 20:43:14 UTC

[drlvm][interpreter] ChoiceFormatTest.test_parseLjava_lang_StringLjava_text_ParsePosition() fails on interpreter

Hello, Ivan!
You wrote,
> why my authorship was discarded from interpreter.cpp?

The authorship is not easily discarded - there are times when others are
painfully trying to recall who have written that code. :-) Could you
please look into the following issue with the interpreter, reported by
Alexey Varlamov?

http://issues.apache.org/jira/browse/HARMONY-1672

Thank you in advance!

With best regards,
Alexei Fedotov,
Intel Middleware Products Division

>-----Original Message-----
>From: Ivan Volosyuk [mailto:ivan.volosyuk@gmail.com]
>Sent: Thursday, July 06, 2006 6:17 PM
>To: harmony-dev@incubator.apache.org
>Subject: Re: svn commit: r419557 - in
>/incubator/harmony/enhanced/drlvm/trunk/vm:
interpreter/src/interpreter.cpp
>vmcore/src/init/vm.cpp vmcore/src/jit/jit_runtime_support.cpp
>vmcore/src/verifier/Verifier.cpp
>
>It is good to see that we finally have java1.5 support in DRLVM.
>One small question, why my authorship was discarded from
interpreter.cpp?
>:)
>--
>Ivan
>
>On 7/6/06, geirm@apache.org <ge...@apache.org> wrote:
>> Author: geirm
>> Date: Thu Jul  6 07:02:51 2006
>> New Revision: 419557
>>
>> URL: http://svn.apache.org/viewvc?rev=419557&view=rev
>> Log:
>> HARMONY-677
>>
>> minimal 1.5 class support - DRLVM can handle v49 classfiles
>>
>>
>> Modified:
>>
>incubator/harmony/enhanced/drlvm/trunk/vm/interpreter/src/interpreter.c
pp
>>     incubator/harmony/enhanced/drlvm/trunk/vm/vmcore/src/init/vm.cpp
>>
>incubator/harmony/enhanced/drlvm/trunk/vm/vmcore/src/jit/jit_runtime_su
ppor
>t.cpp
>>
>incubator/harmony/enhanced/drlvm/trunk/vm/vmcore/src/verifier/Verifier.
cpp
>>
>> Modified:
>incubator/harmony/enhanced/drlvm/trunk/vm/interpreter/src/interpreter.c
pp
>> URL:
>http://svn.apache.org/viewvc/incubator/harmony/enhanced/drlvm/trunk/vm/
inte
>rpreter/src/interpreter.cpp?rev=419557&r1=419556&r2=419557&view=diff
>>
>=======================================================================
====
>===
>> ---
>incubator/harmony/enhanced/drlvm/trunk/vm/interpreter/src/interpreter.c
pp
>(original)
>> +++
>incubator/harmony/enhanced/drlvm/trunk/vm/interpreter/src/interpreter.c
pp
>Thu Jul  6 07:02:51 2006
>> @@ -13,10 +13,6 @@
>>   *  See the License for the specific language governing permissions
and
>>   *  limitations under the License.
>>   */
>> -/**
>> - * @author Ivan Volosyuk
>> - * @version $Revision: 1.61.4.15.4.4 $
>> - */
>>  #include "interpreter.h"
>>  #include "interpreter_exports.h"
>>  #include "interpreter_imports.h"
>> @@ -955,18 +951,22 @@
>>
>>  #ifndef NDEBUG
>>      switch(cp_tag(cp, index)) {
>> -        case 8:
>> +        case CONSTANT_String:
>>              DEBUG_BYTECODE("#" << dec << (int)index << " String: \""
<<
>cp[index].CONSTANT_String.string->bytes << "\"");
>>              break;
>
>---------------------------------------------------------------------
>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