Changeset: 8f61ac5986ee
Author:khazra
Date: 2012-03-02 13:48 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/8f61ac5986ee
7146763: Warnings cleanup in the sun.rmi and related packages
Summary: Cleanup warnings and use jkd7 features in sun.rmi.*
Reviewed-by: smarks, chegar, for
Changeset: ce6b852bf4e2
Author:khazra
Date: 2012-03-05 17:38 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/ce6b852bf4e2
7151348: Build breaks due to warning clean up in sun.rmi.*(7146763)
Summary: Undo changes to sun/rmi/rmic/* commited as fix for 7146763
Reviewed-by: smarks
Changeset: 3bfebedb549f
Author:khazra
Date: 2012-03-15 13:21 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3bfebedb549f
7045655: An empty InMemoryCookieStore should not return true for removeAll
Summary: CookieStore.removeAll() should return false for an empty CookieStore
Re
Changeset: 337d4570b8d6
Author:khazra
Date: 2012-03-16 11:52 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/337d4570b8d6
7152007: Fix warnings in sun/rmi/rmic
Summary: Minor code changes to remove warnings in sun/rmi/rmic
Reviewed-by: chegar, smarks
! src/share/classes/sun/r
Changeset: 31c15e2f51ba
Author:khazra
Date: 2012-04-17 11:59 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/31c15e2f51ba
7152856: TEST_BUG: sun/net/www/protocol/jar/B4957695.java failing on Windows
Summary: Remove usage of HTTP Server at test/sun/net/www/httptest
Reviewed-by:
Changeset: 1757f049e8c0
Author:khazra
Date: 2012-04-17 12:21 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/1757f049e8c0
7157893: Warnings Cleanup in java.util.*
Summary: Minor code changes to cleanup warnings in java.util.*
Reviewed-by: mduigou, naoto, smarks
Contributed-by:
Changeset: bc51d0569ccd
Author:khazra
Date: 2012-04-19 13:26 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/bc51d0569ccd
7162385: TEST_BUG: sun/net/www/protocol/jar/B4957695.java failing again
Summary: Enable finding "foo1.jar"
Reviewed-by: chegar
! test/sun/net/www/protocol
Changeset: 715f50872ae7
Author:khazra
Date: 2012-04-19 18:11 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/715f50872ae7
7158636: InterfaceAddress.getBroadcast() returns invalid broadcast address on
WLAN
Summary: Update Windows native code to infer WLAN interface type in Win
Hi,
This change inserts a null check for the key being passed to
Preferences.remove() on Mac, so that
the method throws a NullPointerException when key is null (according to
its specification).
Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7160242
Webrev: http://cr.openjdk.java.net/
Changeset: fcdbd1f34309
Author:khazra
Date: 2012-04-24 14:59 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/fcdbd1f34309
7144274: [macosx] Default IPv6 multicast interface is not being set when
calling MulticastSocket.joinGroup()
Summary: Get default interface for Mac OS X w
Changeset: 3e398b549cea
Author:khazra
Date: 2012-04-25 12:31 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3e398b549cea
7160242: (prefs) Preferences.remove(null) does not throw NPE [macosx]
Summary: Insert null check of argument in remove()'s implementation
Reviewed-by: fora
Changeset: 108a02a57b75
Author:khazra
Date: 2012-04-26 12:04 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/108a02a57b75
7118100: (prefs) Inconsistency when using system and user preference on OSX Lion
Summary: Enable user to read/write preferences to persistent storage
Revie
Changeset: 59121a4c71c6
Author:khazra
Date: 2012-05-09 11:14 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/59121a4c71c6
7165118: (prefs) AbstractPreferences.remove(null) does not throw NPE
Summary: Insert null argument check in AbstractPreferences.remove()
Reviewed-by: dholm
Changeset: 5152c832745a
Author:khazra
Date: 2012-05-09 16:55 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5152c832745a
7096436: (sc) SocketChannel.connect fails on Windows 8 when channel configured
non-blocking
Summary: Set localAddress only when connection is established
Changeset: 9a18e318f95a
Author:khazra
Date: 2012-05-15 11:51 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/9a18e318f95a
7164636: (prefs) Cleanup src/macosx/classes/java/util/prefs
Summary: Remove rawtype usages and other code cleanup
Reviewed-by: chegar, briangoetz
! src/ma
I ran the tests in jdk/test/java/lang and they all pass. Let me know if
you think some other tests should also
be run.
- Kurchi
On 5/22/12 7:33 PM, David Holmes wrote:
On 23/05/2012 11:32 AM, Kurchi Hazra wrote:
Hi,
This is a minor change to java_props_md.c to return "Windows 8" as the
"os.n
Changeset: 0c3d9050c918
Author:khazra
Date: 2012-05-23 10:41 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/0c3d9050c918
7170169: (props) System.getProperty("os.name") should return "Windows 8" when
run on Windows 8
Summary: Enable Windows Version 6.2 to be recognized as Win
Changeset: eb441933f6fe
Author:khazra
Date: 2012-05-29 13:16 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/eb441933f6fe
7171591: getDefaultScopeID() in src/solaris/native/java/net/net_util_md.c
should return a value
Summary: Use CHECK_NULL_RETURN instead of CHECK_NULL
Revie
Changeset: af313ded4ffb
Author:khazra
Date: 2012-06-06 11:37 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/af313ded4ffb
7173645: (props) System.getProperty("os.name") should return "Windows Server
2012" for Windows Server 2012
Summary: Enable Windows Server 2012 to be recog
Hi,
I was just going through the patches, there are some more fallthrough
cases in
src/share/classes/java/util/regex/Pattern.java.(for example in line 2247).
Are these not generating warnings?
- Kurchi
On 6/20/12 7:30 AM, Martijn Verburg wrote:
Hi all,
Apologies, I didn't check that atta
I used a new workspace and missed adding it to mercurial. The test
remains the same,
I'll push it after adding the test.
- Kurchi
On 7/12/12 12:16 AM, Alan Bateman wrote:
On 12/07/2012 00:45, Kurchi Hazra wrote:
On 7/11/12 4:24 PM, Chris Hegarty wrote:
On 12 Jul 2012, at 00:15, Kurchi
Hazra
Changeset: e9461aeff91f
Author:khazra
Date: 2012-07-13 16:02 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e9461aeff91f
7160252: (prefs) NodeAddedEvent was not delivered when new node add when new
Node
Summary: Change native code to convey to Java code whether a new node wa
Hi,
Some tests in our test repository assume that
InetAddress.getLocalHost() will return 127.0.0.1. However, with
linux systems running Ubuntu 12.04 now returning 127.0.1.1 as localhost,
these tests are failing. I have changed two
of the NIO tests to remove their dependency on
InetAddress.g
Changeset: 1469be7182b4
Author:khazra
Date: 2012-07-16 16:30 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/1469be7182b4
7177045: Rework the TestProviderLeak.java regression test, it is too fragile to
low memory errors.
Summary: Increase Xmx to 20 MB and add mechanisms to ea
Changeset: 773474da570b
Author:khazra
Date: 2012-07-18 15:19 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/773474da570b
7185051: Remove TestProviderLeak.java from the ProblemList
Summary: Remove TestProviderLeak.java from jdk test problem list.
Reviewed-by: khazra
Contribute
Hi,
While flushing user preferences, a BackingStoreException is not expected, since
the current user should be able to write such preferences to the
persistent storage (unlike system preferences, where current user needs
to have admin rights). This fix aims at allowing a user preference node
Thanks Alan. There is another bug on synchronization in the prefs
implementation that I need to look into.
I will get rid of the extra brace, and push this then.
- Kurchi
On 7/22/12 12:16 PM, Alan Bateman wrote:
On 19/07/2012 19:31, Kurchi Subhra Hazra wrote:
Hi,
While flushing user
Changeset: 74ceda3a98a0
Author:khazra
Date: 2012-07-24 13:38 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/74ceda3a98a0
7184287: (prefs) BackingStoreException when calling flush on root node[macosx]
Summary: Change implementation to enable user without administrative privile
Changeset: bd91a601265c
Author:khazra
Date: 2012-08-24 11:48 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/bd91a601265c
7168172: (fs) Files.isReadable slow on Windows
Summary: Remove DACL checking for read access, also reviewed by
[email protected], [email protected]
R
Thanks for cleaning up those spaces Dan. The changes look fine.
Sorry for the extra trouble!
- Kurchi
On 8/28/12 10:22 PM, Dan Xu wrote:
It is funny. :) I have searched all source codes under jdk and removed
spaces for the similar cases.
Please review the new version of change at,
http://cr.
Changeset: e143d8f8e477
Author:dxu
Date: 2012-09-18 14:45 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e143d8f8e477
7195933: There is incorrect link to "Info-ZIP Application Note 970311" in doc
page of Package java.util.zip
Summary: Correct a java doc link in java.util.zip
On 9/29/12 2:23 PM, Ulf Zibis wrote:
Am 29.09.2012 22:52, schrieb Alan Bateman:
http://cr.openjdk.java.net/~alanb/8000269/webrev/index.html
Class.java :
You mean java.io.FilePermission.java?
- in lines 702, 825 indentation should be 8.
- in line 826 I guess you wanted to move the curly br
On 10/12/12 12:43 AM, Alan Bateman wrote:
On 12/10/2012 01:21, Kurchi Hazra wrote:
Hi,
This is a regression introduced by the fix for 7160252[1] that I
pushed a few weeks ago. We should
really be using the class member field isUser, rather than the
argument passed in the constructor as a p
On 10/16/12 2:54 AM, Alan Bateman wrote:
On 16/10/2012 01:53, Kurchi Hazra wrote:
Hi Alan,
Please find an updated webrev with a test included:
http://cr.openjdk.java.net/~khazra/7198073/webrev.01/
CheckUserPrefsStorage.sh is the main entry point - it runs
CheckUserPrefFirst first (which
Changeset: 3a6b76a468bd
Author:khazra
Date: 2012-10-16 15:23 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3a6b76a468bd
7198073: (prefs) user prefs not saved [macosx]
Summary: Using class member field to get node instead of argument
Reviewed-by: alanb
! src/macosx/classes/j
Hi,
The tests in jdk/test/util/prefs are not good candidates to run concurrently,
since on platforms such as Windows and Mac OS X, they currently read/write from
the same location on the persistent storage. This fix simply informs jtreg[1] to
not run these tests concurrently when using the co
Updated webrev: http://cr.openjdk.java.net/~khazra/8003518/webrev.01/
- Kurchi
On 11/15/12 10:28 PM, Alan Bateman wrote:
On 16/11/2012 06:16, Kurchi Subhra Hazra wrote:
Hi,
The tests in jdk/test/util/prefs are not good candidates to run
concurrently,
since on platforms such as Windows
Changeset: 0ee09f17361e
Author:khazra
Date: 2012-11-16 12:28 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/0ee09f17361e
8003518: (prefs) Tests in jdk/test/java/util/prefs should not be run
concurrently
Summary: Add java/util/prefs to exclusiveAccess.dirs in TEST.ROOT
Review
Changeset: 43d2e02c4098
Author:khazra
Date: 2012-11-30 12:00 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/43d2e02c4098
7197662: (prefs) java/util/prefs/AddNodeChangeListener.java fails by timeout or
by "couldn't get file lock"
Summary: Set -Djava.util.prefs.userRoot to cur
Changeset: e8414d69692c
Author:khazra
Date: 2013-01-17 14:50 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e8414d69692c
7171415: java.net.URI.equals/hashCode not consistent for some URIs
Summary: Rewrite URI.hashCode() to consider encoded characters, also reviewed
by vita..
Changeset: 77bde15bc6a9
Author:khazra
Date: 2013-01-25 11:52 -0800
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/77bde15bc6a9
7017962: Obsolete link is used in URL class level spec
Summary: Change the link to an archived document
Reviewed-by: chegar, mduigou
! src/share/classes/j
Changeset: b702977e7212
Author:khazra
Date: 2013-04-05 12:12 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/b702977e7212
5001942: Missings SOCKS support for direct connections
Summary: Add support for socksNonProxyHosts
Reviewed-by: chegar, khazra
Contributed-by: Christos Zou
Forwarding to core-libs.
- Kurchi
Original Message
Subject:Latency in starting threads on Mac OS X
Date: Tue, 16 Apr 2013 15:57:06 +0300
From: Dr Heinz M. Kabutz
Organization: JavaSpecialists.eu
To: [email protected]
Good day my fellow Mac O
Changeset: 3062bf908281
Author:khazra
Date: 2013-05-02 14:26 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3062bf908281
8013140: Heap corruption with NetworkInterface.getByInetAddress() and long i/f
name
Summary: Remove buffer overruns in native code
Reviewed-by: alanb, che
Changeset: 86c1e8c799f5
Author:khazra
Date: 2013-05-13 13:48 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/86c1e8c799f5
8014254: Selector in HttpServer introduces a 1000 ms delay when using KeepAlive
Summary: Rearrange event-handling code to remove bottle-neck. Also reviewed
Changeset: 790d292ee761
Author:khazra
Date: 2013-05-14 12:01 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/790d292ee761
6328537: Improve javadocs for Socket class by adding references to SocketOptions
Summary: Insert references to SocketOptions.java where applicable
Reviewed
Changeset: a8be9405bb4b
Author:khazra
Date: 2013-05-16 10:58 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/a8be9405bb4b
7150552: network test hangs [macosx]
Summary: Remove usage of test/sun/net/www/httptest
Reviewed-by: chegar
! test/ProblemList.txt
! test/java/net/CookieH
On 5/21/13 1:35 AM, Mandy Chung wrote:
Kurchi,
The windows implementation also emits warning messages that should be
fixed up too since we don't want to emit anything to interfere the
application logging. The default logging configuration (default level
is INFO) should apply to the applicati
On 5/21/13 9:40 AM, Mike Duigou wrote:
Any chance that it would be possible to stop the periodic attempts to sync
preferences?
- I did think of that too - but the syncing may fail(temporarily) if the
prefs object could not retrieve the lock to the
associated prefs file. We could stop syncing
On 5/21/13 9:53 AM, Kurchi Subhra Hazra wrote:
On 5/21/13 9:40 AM, Mike Duigou wrote:
Any chance that it would be possible to stop the periodic attempts to
sync preferences?
- I did think of that too - but the syncing may fail(temporarily) if
the prefs object could not retrieve the lock to
Changeset: 11cdcf87ad5d
Author:jzavgren
Date: 2013-05-31 15:23 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/11cdcf87ad5d
8008972: Memory leak: Java_java_net_TwoStacksPlainDatagramSocketImpl_receive0
[parfait]
Summary: Modified the code so that "jumbo frames" are truncated
Changeset: 8b65dfe8f509
Author:khazra
Date: 2013-06-07 10:59 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/8b65dfe8f509
7051862: CookiePolicy spec conflicts with CookiePolicy.ACCEPT_ORIGINAL_SERVER
Summary: Return false for null arguments in
ACCEPT_ORIGINAL_SERVER#shouldAcc
Changeset: ff83bd43e36a
Author:khazra
Date: 2013-06-13 11:23 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/ff83bd43e36a
8015421: NegativeArraySizeException occurs in ChunkedOutputStream() with
Integer.MAX_VALUE
Summary: Ensure integer overflow does not occur
Reviewed-by: ch
Changeset: 42f9ad39bf42
Author:khazra
Date: 2013-06-13 17:48 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/42f9ad39bf42
7169142: CookieHandler does not work with localhost
Summary: Add .local to derived effective hostnames without dot
Reviewed-by: chegar
! src/share/classes
Changeset: de6b93fd6d23
Author:khazra
Date: 2013-06-19 14:02 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/de6b93fd6d23
8016576: Overrides warnings in jdi and jconsole
Summary: Implement hashCode() in classes emitting warnings
Reviewed-by: alanb, chegar
! src/share/classes/
Changeset: e1b18a666f76
Author:khazra
Date: 2013-06-19 14:13 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e1b18a666f76
8016698: Cleanup overrides warning in sun/tools/ClassDeclaration.java
Summary: Override Object.hashCode()
Reviewed-by: alanb, chegar
! src/share/classes/s
Changeset: 2b156531b7eb
Author:arieber
Date: 2013-06-19 17:41 -0700
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/2b156531b7eb
7025238: HttpURLConnection does not handle URLs with an empty path component.
Summary: Prepend a '/' to file when path is empty
Reviewed-by: chegar, khazr
Changes look good to me (once Alan's point is verified).
Thanks,
- Kurchi
On Wed, Jul 31, 2013 at 7:15 AM, Alan Bateman wrote:
>
> The changes in the webrev look okay to me but the reference to the "app
> compatibility shim" in the MS article is a bit confusing and not clear to
> me (with chec
58 matches
Mail list logo
