th Harmony-1942 applied?
On 10/27/06, Vladimir Strigun <[EMAIL PROTECTED]> wrote:
>
> Mikhail,
>
> Not yet. As I mentioned in the thread I'm still working on Dacapo.
> I'll let you know if I find any improvements for JIT.
>
> Thanks,
> Vladimir.
>
> O
On 10/27/06, Geir Magnusson Jr. <[EMAIL PROTECTED]> wrote:
This is a nice note to wake up to...
Vladimir Strigun wrote:
> Here are the results:
>
> Small workload:
> OrigBuildFixedSun1.5.0_06
> bloat996,0781024,85955,589
> chart1240,7771
VM) compared to Sun 1.5
> using decapo?
>
> geir
>
>
> Vladimir Strigun wrote:
> > The optimization covers the following issues:
> > - java.nio.charset.CharsetDecoder and java.nio.charset.CharsetEncoder
> > Streaming decoding/encoding was removed. Analysis of API hots
f read(char[], int,
> int) uses several invocation of System.arraycopy. Proposed solution
> wraps char[] arguments within char buffer and therefore doesn't use
> arraycopy. Decoding operation is also produced inside the method, so
> fillBuf() has been removed
>
> Thoughts? Comment
char buffer and therefore doesn't use
arraycopy. Decoding operation is also produced inside the method, so
fillBuf() has been removed
Thoughts? Comments?
Thanks,
Vladimir.
On 10/26/06, Vladimir Strigun (JIRA) <[EMAIL PROTECTED]> wrote:
[classlib][performance] performance improvement f
Mikhail,
How do you solve the problem?
I have the same error on my machine (Win XP SP2).
Thanks,
Vladimir.
On 8/9/06, Mikhail Loenko <[EMAIL PROTECTED]> wrote:
Seems like it isn't related to the new VM - I see similar error message
on "svn move".
But the problem with running the tests disappe
shows
significantly better performance for BigDecimal that fit in 64 bits
with no harm for the remaining cases.
I agree with the changes you have made and are willing to continue
improving our common math from now on.
Thanks a lot!
Daniel
On 8/1/06, Vladimir Strigun <[EMAIL PROTECTE
On 8/2/06, Geir Magnusson Jr <[EMAIL PROTECTED]> wrote:
Vladimir Strigun wrote:
[SNIP]
> JavaWorld Benchmark - benchmark for low-level operations: loops,
> accessing variables, method invocation, arithmetic operators, casting,
> instantiation, exception handling, thre
On 8/2/06, Geir Magnusson Jr <[EMAIL PROTECTED]> wrote:
Vladimir Strigun wrote:
> Our project is now in a good shape and we started encouraging people
> to try real applications on top of Harmony. One of the important parts
> of the feedback will be general impression of stabili
pecJVM, SpecJBB, SpecJAppserver. Unfortunately first we should get
license for it, but I believe this issue can be solved within the help
of companies participating in Harmony :)
Thoughts? Comments?
[1] http://dacapobench.org
[2] http://www.spec.org
--
Vladimir Strigun,
Intel Middleware Pr
oost for BigDecimal that
fits to 64 bits. I should mention that we can't see any degradation in
all other performance tests with our patch.
Daniel, could you please review our patch? If you agree with suggested
changes, I believe we all will vote +1 for our common math :)
Thanks,
Vladimir.
eed. What I suggested is calculate "golden" values on the fly, so you'll
have golden values for the given machine, VM, etc
Anyway if you feel the tests should not go to any suite, I'll just
close the JIRA issue.
Thanks,
Mikhail
2006/6/23, Vladimir Strigun <[EMAIL PROTEC
den' performances and than measure where the current performance
in comparison to the golden is.
This scenario might be simplified. If your optimization works e.g. for numbers
less than say 1'000'000, you can compare performance for 999'999 and
1'000'001
Thanks
rification to the conventions
> proposal.
>
> Thanks,
> Mikhail
>
> 2006/6/20, Vladimir Strigun <[EMAIL PROTECTED]>:
> > On 6/20/06, Mikhail Loenko <[EMAIL PROTECTED]> wrote:
> > > I think they are not unit tests and should go to a different
> > >
sting convention.
What do you think about /src/tests/performance ?
Thanks,
Vladimir.
Thanks,
Mikhail
2006/6/20, Vladimir Strigun <[EMAIL PROTECTED]>:
> Hi Mikhail,
>
> AFAIK, we haven't such kind of tests in svn yet. It's hard to define
> best place for it, but since th
n the HARMONY-551 are?
Thanks,
Mikhail
2006/6/2, Vladimir Strigun <[EMAIL PROTECTED]>:
> Our team has done some analysis of current Harmony implementation of
> java.math package. The implementation was considered from the
> performance point of view and I'd like to share results
ing, etc.
Within these type of application values usually fit to 32 bit, that's
why we have special case for small numbers. Also, these optimizations
do not degrade in common.
Thanks,
Vladimir.
Thanks again,
Daniel
On 6/2/06, Vladimir Strigun <[EMAIL PROTECTED]> wrote:
> Our team has
ady have been
implemented in ITC contribution it will be great to compare internal
representation of BigInteger and try to measure affect of different
approaches.
On 6/2/06, Vladimir Strigun (JIRA) <[EMAIL PROTECTED]> wrote:
[ http://issues.apache.org/jira/browse/HARMONY-551?page=
'd like to document this behaviour.
Thanks.
Vladimir.
What's your opnion?
Thanks.
On 5/15/06, Vladimir Strigun <[EMAIL PROTECTED]> wrote:
>
> On 5/5/06, Mikhail Loenko <[EMAIL PROTECTED]> wrote:
> > 2006/5/5, Vladimir Strigun <[EMAIL PROTECTED]>:
> &g
On 5/5/06, Mikhail Loenko <[EMAIL PROTECTED]> wrote:
2006/5/5, Vladimir Strigun <[EMAIL PROTECTED]>:
> On 5/5/06, Mikhail Loenko <[EMAIL PROTECTED]> wrote:
> > Vladimir, Andrew
> >
> > 2006/4/26, Andrew Zhang <[EMAIL PROTECTED]>:
> > >
://bugs.icu-project.org/cgi-bin/icu-bugs/conversion?id=5074)
On 5/4/06, Vladimir Strigun (JIRA) <[EMAIL PROTECTED]> wrote:
>
>[
> http://issues.apache.org/jira/browse/HARMONY-33?page=comments#action_12377817]
>
> Vladimir
ker is more
> confusable and may think:
>
> "Is the remaining bytebuffer maintained in decoder or in ? Shall I get the
> remaining buffer from in and pass it for next invocation?"
>
> Anyway, we need a decision on this compatibility issue.
> By the wa
On 4/28/06, Paulex Yang <[EMAIL PROTECTED]> wrote:
Richard Liang wrote:
> Paulex Yang wrote:
>> I see, the OutputStreamWriterTest does fail, seems there's some
>> problem for CharsetDecoder to handling UTF-8 byte stream, but the
>> InputStreamReader itself should work because it pass all tests fo
On 4/26/06, Paulex Yang <[EMAIL PROTECTED]> wrote:
> Andrew Zhang wrote:
> >
> > Mikhail Loenko wrote:
> >> 2006/4/26, Andrew Zhang <[EMAIL PROTECTED]>:
> >>> Vladimir Strigun wrote:
> >>>> On 4/26/06, Andrew Zhang <[EMAIL PROTECT
ning() for calculating endOfInput. In this case, we always
pass false with first invocation of decode method. Algorithm further
is almost the same as above, but we stop the cycle if all bytes
decoded successfully (i.e if bytes.hasRemaining() is false).
What do you think about it?
Thanks.
Vladimir.
>
w many bytes are available and you have to return 0.
>
> So I think the patch should be modified to avoid 'available()'
>
> Thanks,
> Mikhail
>
> >-Original Message-
> >From: Vladimir Strigun (JIRA) [mailto:[EMAIL PROTECTED]
> >Sent: Monday, Apr
file to remind
> > us how to do it).
> >
> > Regards,
> > Tim
> >
> > Matt Benson wrote:
> > > again, guys, to be sure it's not missed:
> > >
> > > adding "-Dhttp.proxyHost=proxy -Dhttp.proxyPort=8080"
> > > to the AN
Thanks Tim, everything works fine.
Thanks,
Vladimir.
On 3/16/06, Tim Ellison <[EMAIL PROTECTED]> wrote:
> Exactly, thanks Vladimir. I've put that in at repo revision 386377
> (with slightly modified property names).
>
> Let me know if you need more.
>
> Thanks,
>
d them into the right
> structure manually and avoid all this.
>
> Regards,
> Tim
>
> Vladimir Strigun wrote:
> > Mikhail,
> >
> > You can use ant setproxy[1] task for resolving the problem. It works
> > for me. On the other hand, I don't think we shoul
Mikhail,
You can use ant setproxy[1] task for resolving the problem. It works
for me. On the other hand, I don't think we should patch build files.
Perhaps, It will be useful to document this somewhere.
Thanks,
Vladimir.
[1] http://ant.apache.org/manual/OptionalTasks/setproxy.html
On 3/16/06, M
[ http://issues.apache.org/jira/browse/HARMONY-166?page=all ]
Vladimir Strigun updated HARMONY-166:
-
Attachment: InputStreamReaderTest.java
test for this issue. it's the same as in description, but it's more comfortable
to read.
> met
[ http://issues.apache.org/jira/browse/HARMONY-166?page=all ]
Vladimir Strigun updated HARMONY-166:
-
Attachment: InputStreamReader.patch.txt
Please try my patch.
> method read() in InputStreamReader failed to read one charac
[
http://issues.apache.org/jira/browse/HARMONY-166?page=comments#action_12368729
]
Vladimir Strigun commented on HARMONY-166:
--
The cause of problem is the following: if decode operation returns underflow,
and the input have available bytes we
: Vladimir Strigun
I've started to play with harmony-57 contribution and found bug in
InputStreamReader class. Method read() should read a single character from
input stream but it works incorrectly for 2 bytes-per-char charsets. Example
below shows that it failed to read one character in U
[
http://issues.apache.org/jira/browse/HARMONY-137?page=comments#action_12368301
]
Vladimir Strigun commented on HARMONY-137:
--
Tim, I agree with the resolution, please close it.
> CharsetDecoder should replace undefined bytes with replacem
Components: Classlib
Reporter: Vladimir Strigun
Priority: Minor
Looks like characters 0x0384 and 0xB7 are unmapped in ICU implementation.
Mapping table could be found here:
http://www.unicode.org/Public/MAPPINGS/ISO8859/8859-7.TXT
Please use test below for reproducing:
import java.nio
Reporter: Vladimir Strigun
Priority: Minor
Corresponding to cp1250 mapping table, 0x81 byte is undefined. See
http://www.unicode.org/Public/MAPPINGS/VENDORS/MICSFT/WINDOWS/CP1250.TXT
So, charset decoder should replace undefined bytes with default replacement,
i.e. 0xFFFD.
Testcase for
[
http://issues.apache.org/jira/browse/HARMONY-29?page=comments#action_12367824 ]
Vladimir Strigun commented on HARMONY-29:
-
Sorry, I meant Util.decode(fileString, false)
> java.util.zip.ZipException while onening jar file on local mach
[
http://issues.apache.org/jira/browse/HARMONY-29?page=comments#action_12367823 ]
Vladimir Strigun commented on HARMONY-29:
-
Since path can contain "+" character, second parameter for decode method should
be false, i.e. Util.decode(fileSt
[
http://issues.apache.org/jira/browse/HARMONY-40?page=comments#action_12367818 ]
Vladimir Strigun commented on HARMONY-40:
-
Thanks, Tim. Can't reproduce with latest sources.
> FileChannel assotiated with FileOutputStream not closed after
On 2/22/06, Mikhail Loenko <[EMAIL PROTECTED]> wrote:
> I'd suggest splitting the issue into two different ones:
>
> The first one is interpreting the command line. I completely agree
> that we should
> do it similar to RI.
>
> The second one is as Vladimir said that behavior is caused by something
This issue can be fixed either in Harmony classlib or kernel classes in VM.
Thanks,
Vladimir Strigun,
Intel Middleware Products Division
On 2/21/06, Alexey Petrenko <[EMAIL PROTECTED]> wrote:
> IBM VM?
>
> If so it's probably not an issue for Harmony but for IBM :)
>
IMO problem is inside VM kernel classes.
What do you think we should do with this issue?
[1] http://java.sun.com/j2se/1.4.2/docs/tooldocs/windows/classpath.html
Thanks,
Vladimir Strigun,
Intel Middleware Products Division
[
http://issues.apache.org/jira/browse/HARMONY-70?page=comments#action_12367017 ]
Vladimir Strigun commented on HARMONY-70:
-
Look like this issue already have been fixed in Harmony-42. I can't reproduce
it with the latest so
[
http://issues.apache.org/jira/browse/HARMONY-102?page=comments#action_12366886
]
Vladimir Strigun commented on HARMONY-102:
--
Here is updated testcase:
import junit.framework.*;
import java.util.Date;
public class DateTest extends TestCase
[ http://issues.apache.org/jira/browse/HARMONY-102?page=all ]
Vladimir Strigun updated HARMONY-102:
-
Attachment: fix102.txt
Fix for the bug. It also include fix for IllegalArgumentException for next
string: "00:00:00 GMT +0130 Sat, 11 Jan
[
http://issues.apache.org/jira/browse/HARMONY-67?page=comments#action_12366469 ]
Vladimir Strigun commented on HARMONY-67:
-
I don't think that this bug should be reopened. Possibly it's another
"compatibility" bug, and I&#
[
http://issues.apache.org/jira/browse/HARMONY-33?page=comments#action_12366344 ]
Vladimir Strigun commented on HARMONY-33:
-
Tim, thank you, I can't reproduce it with latest sources.
> java.nio.BufferOverflow exception while decoding By
[
http://issues.apache.org/jira/browse/HARMONY-40?page=comments#action_12366330 ]
Vladimir Strigun commented on HARMONY-40:
-
This bug is a part of Harmony-42 issue. I can't reproduce it with latest
sources. Anyway, I think suggested test c
[
http://issues.apache.org/jira/browse/HARMONY-29?page=comments#action_12365891 ]
Vladimir Strigun commented on HARMONY-29:
-
another fix (using internal utils):
37a38
> import com.ibm.oti.util.Util;
235c236
< jar = new J
On 2/10/06, Stefano Mazzocchi <[EMAIL PROTECTED]> wrote:
> Vladimir Strigun wrote:
> > I'm able to run eclipse on Harmony classlib + J9 VM. To run Eclipse
> > you need to make following steps:
> >
> > - download xalan.jar, xercesImpl.jar, xml-apis.jar and seri
[
http://issues.apache.org/jira/browse/HARMONY-67?page=comments#action_12365756 ]
Vladimir Strigun commented on HARMONY-67:
-
Looks like duplicate of Harmony-33
> java.nio.charset.Charset.decode(ByteBuffer) throws unexpec
On 2/9/06, Tim Ellison <[EMAIL PROTECTED]> wrote:
> Vladimir Strigun wrote:
> > I'm able to run eclipse on Harmony classlib + J9 VM. To run Eclipse
> > you need to make following steps:
> >
> > - download xalan.jar, xercesImpl.jar, xml-apis.jar and serializer.
I'm able to run eclipse on Harmony classlib + J9 VM. To run Eclipse
you need to make following steps:
- download xalan.jar, xercesImpl.jar, xml-apis.jar and serializer.jar
from xml.apache.org and put it to deploy/jre/lib/boot/
- add next line to jre/lib/boot/bootclasspath.properties :
bootclasspa
[
http://issues.apache.org/jira/browse/HARMONY-71?page=comments#action_12365225 ]
Vladimir Strigun commented on HARMONY-71:
-
Trivial fix for the issue:
@@ -920,7 +920,7 @@
*/
public void setUseCaches(boolean newValue
[
http://issues.apache.org/jira/browse/HARMONY-72?page=comments#action_12365218 ]
Vladimir Strigun commented on HARMONY-72:
-
Fix is trivial:
@@ -733,7 +733,7 @@
*/
public void setAllowUserInteraction(boolean newValue
[
http://issues.apache.org/jira/browse/HARMONY-45?page=comments#action_12364478 ]
Vladimir Strigun commented on HARMONY-45:
-
I'm ok with this fix.
> Charset.isRegistered() method always ret
[
http://issues.apache.org/jira/browse/HARMONY-24?page=comments#action_12364368 ]
Vladimir Strigun commented on HARMONY-24:
-
Thanks Tim, it's fixed.
> java.net.URLEncoder.encode(String s, String enc) doesn't throw
> UnsupportedE
[
http://issues.apache.org/jira/browse/HARMONY-52?page=comments#action_12364201 ]
Vladimir Strigun commented on HARMONY-52:
-
Ive found small testcase for the issue:
public static void main(String[] args) throws Exception{
byte[] arr
[
http://issues.apache.org/jira/browse/HARMONY-45?page=comments#action_12363988 ]
Vladimir Strigun commented on HARMONY-45:
-
Patch is trivial (just check whether the name of charset starts with "x-"):
567c567
<
[
http://issues.apache.org/jira/browse/HARMONY-45?page=comments#action_12363987 ]
Vladimir Strigun commented on HARMONY-45:
-
Testcase:
package org.apache.harmony.tests.java.nio.charset;
import java.nio.charset.Charset;
import
Charset.isRegistered() method always return true
Key: HARMONY-45
URL: http://issues.apache.org/jira/browse/HARMONY-45
Project: Harmony
Type: Bug
Components: Classlib
Reporter: Vladimir Strigun
Priority
[
http://issues.apache.org/jira/browse/HARMONY-40?page=comments#action_12363705 ]
Vladimir Strigun commented on HARMONY-40:
-
Forced close of file current file channel in file output stream can be added
(diff for current FileOutputStream
[
http://issues.apache.org/jira/browse/HARMONY-40?page=comments#action_12363701 ]
Vladimir Strigun commented on HARMONY-40:
-
I can't attach testcase, so posting it as a comment:
/* Copyright 2005 The Apache Software Foundation or its licensor
Components: Classlib
Reporter: Vladimir Strigun
When I receive FileChannel from file output stream, write something to stream
and then close it, channel, assotiated with the stream is still open. I'll
attach unit test for the issue.
--
This message is automatically generated by
[
http://issues.apache.org/jira/browse/HARMONY-37?page=comments#action_12363586 ]
Vladimir Strigun commented on HARMONY-37:
-
Tim, thanks for the fix. Everything works fine.
> remove() method of IdentityHashMap works incorrec
[ http://issues.apache.org/jira/browse/HARMONY-37?page=all ]
Vladimir Strigun updated HARMONY-37:
Attachment: IdentityHashMapTest.java
test was updated
> remove() method of IdentityHashMap works incorrec
[
http://issues.apache.org/jira/browse/HARMONY-37?page=comments#action_12363468 ]
Vladimir Strigun commented on HARMONY-37:
-
Ya, I've already found it. I'll update test and try to update the fix :)
> remove() method of Identity
[
http://issues.apache.org/jira/browse/HARMONY-37?page=comments#action_12363414 ]
Vladimir Strigun commented on HARMONY-37:
-
To fix it, we just need to check wheather this key exist in hashmap (i.e check
that key is not null and key exist in
[ http://issues.apache.org/jira/browse/HARMONY-37?page=all ]
Vladimir Strigun updated HARMONY-37:
Attachment: IdentityHashMapTest.java
regression test
> remove() method of IdentityHashMap works incorrec
[
http://issues.apache.org/jira/browse/HARMONY-37?page=comments#action_12363416 ]
Vladimir Strigun commented on HARMONY-37:
-
I'll attach regression test for this bug. Here is diff for
/incubator/harmony/enhanced/classlib/trunk/modules/luni/src
remove() method of IdentityHashMap works incorrectly
Key: HARMONY-37
URL: http://issues.apache.org/jira/browse/HARMONY-37
Project: Harmony
Type: Bug
Components: Classlib
Reporter: Vladimir Strigun
When
[
http://issues.apache.org/jira/browse/HARMONY-24?page=comments#action_12363269 ]
Vladimir Strigun commented on HARMONY-24:
-
Current implementation of encode(String, String) method use getBytes(encoding)
method for encoding "special" char
[
http://issues.apache.org/jira/browse/HARMONY-23?page=comments#action_12363266 ]
Vladimir Strigun commented on HARMONY-23:
-
it's ok.
> java.net.URI(String s) for invalid escaped characters throw
> IllegalArgume
[
http://issues.apache.org/jira/browse/HARMONY-21?page=comments#action_12363265 ]
Vladimir Strigun commented on HARMONY-21:
-
Sorry, of course it's ok .
> File constructor produces wrong path to absolute path specified relative to
On 1/19/06, Anthony Green <[EMAIL PROTECTED]> wrote:
> On Wed, 2006-01-18 at 21:51 +0300, Vladimir Strigun wrote:
> > Attached document contains a proposal describing an approach which is
> > trying to enforce portability of nio socket channels implementation by
> > mov
[
http://issues.apache.org/jira/browse/HARMONY-33?page=comments#action_12363090 ]
Vladimir Strigun commented on HARMONY-33:
-
Hi Paulex,
I can't reproduce IllegalStateException with my fix and with next test:
import java.nio.*;
i
[
http://issues.apache.org/jira/browse/HARMONY-33?page=comments#action_12363005 ]
Vladimir Strigun commented on HARMONY-33:
-
The reason of the issue is invocation of put(replacement) method for CharBuffer
with zero capacity. First, during decoding
Components: Classlib
Reporter: Vladimir Strigun
Assigned to: Geir Magnusson Jr
Priority: Minor
If I try to decode ByteBuffer with lengh 1 using UTF-16 decoder unexpected
java.nio.BufferOverflow exception occured. Please see testcase below.
import java.nio.*;
import java.nio.charset
[
http://issues.apache.org/jira/browse/HARMONY-29?page=comments#action_12362663 ]
Vladimir Strigun commented on HARMONY-29:
-
The cause of problem is creation of file with undecoded file path inside
openJarFile method. The fix is very simple and so
Reporter: Vladimir Strigun
Assigned to: Geir Magnusson Jr
When I try to get entries from jar file I get java.util.zip.ZipException. Here
is testcase for reproducing the issue:
import java.net.JarURLConnection;
import java.net.URL;
import java.util.Enumeration;
public class TestZipFile {
public
rmony
Type: Bug
Components: Classlib
Reporter: Vladimir Strigun
Assigned to: Geir Magnusson Jr
Corresponding to API specification method encode(String, String) of
java.net.URLEncoder should throw "UnsupportedEncodingException - If the named
encoding is not supported".
k the natural place for the regression tests should be together
with the unit tests. I am not sure, however, if the regression tests
should be marked explicitly or differ from the unit tests in any other
specific way.
What do you think about it? Any preferences?
Thank you,
Vladimir Strigun, Intel Middl
[
http://issues.apache.org/jira/browse/HARMONY-21?page=comments#action_12361427 ]
Vladimir Strigun commented on HARMONY-21:
-
Unfortunately, I have regression with my fix. Next test is failed with my fix:
import java.io.*;
public class Test
[
http://issues.apache.org/jira/browse/HARMONY-23?page=comments#action_12360570 ]
Vladimir Strigun commented on HARMONY-23:
-
The cause of the IllegalArgument exception is creating of URISyntaxException
with index = -2.
Here is a part of spec for
Components: Classlib
Reporter: Vladimir Strigun
Assigned to: Geir Magnusson Jr
Priority: Minor
When I use URI(String) constructor with invalid escaped characters I get
IllegalArgumentException instead of URISyntaxException.
Small testcase for reproducing this issue:
import
[
http://issues.apache.org/jira/browse/HARMONY-21?page=comments#action_12359731 ]
Vladimir Strigun commented on HARMONY-21:
-
I've found that my fix isn't good for Linux. When I try to create file using
File("//usr") I get inc
[
http://issues.apache.org/jira/browse/HARMONY-21?page=comments#action_12359243 ]
Vladimir Strigun commented on HARMONY-21:
-
The cause of problem is realization of fixSlashes() method of java.io.File.
Another testcase for this bug is almost the
88 matches
Mail list logo
