hg: jdk7/build/langtools: 7033660: Update copyright year to 2011 on any files changed in 2011

2011-04-06 Thread kelly . ohair
Changeset: 0ff2bbd38f10 Author:ohair Date: 2011-04-06 20:33 -0700 URL: http://hg.openjdk.java.net/jdk7/build/langtools/rev/0ff2bbd38f10 7033660: Update copyright year to 2011 on any files changed in 2011 Reviewed-by: dholmes ! make/netbeans/langtools/build.xml ! make/tools/GenStubs

hg: jdk7/build/jdk: 7033660: Update copyright year to 2011 on any files changed in 2011

2011-04-06 Thread kelly . ohair
Changeset: 272483f6650b Author:ohair Date: 2011-04-06 22:06 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jdk/rev/272483f6650b 7033660: Update copyright year to 2011 on any files changed in 2011 Reviewed-by: dholmes ! make/Makefile ! make/com/sun/crypto/provider/Makefile ! make/

hg: jdk7/build/jaxws: 7033660: Update copyright year to 2011 on any files changed in 2011

2011-04-06 Thread kelly . ohair
Changeset: de11bd049d6f Author:ohair Date: 2011-04-06 20:16 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jaxws/rev/de11bd049d6f 7033660: Update copyright year to 2011 on any files changed in 2011 Reviewed-by: dholmes ! build-defs.xml ! build-drop-template.xml ! build.xml ! make

hg: jdk7/build/jaxp: 7033660: Update copyright year to 2011 on any files changed in 2011

2011-04-06 Thread kelly . ohair
Changeset: bc701b263f16 Author:ohair Date: 2011-04-06 20:15 -0700 URL: http://hg.openjdk.java.net/jdk7/build/jaxp/rev/bc701b263f16 7033660: Update copyright year to 2011 on any files changed in 2011 Reviewed-by: dholmes ! build-defs.xml ! build-drop-template.xml ! build.xml ! jaxp.

hg: jdk7/build: 7033660: Update copyright year to 2011 on any files changed in 2011

2011-04-06 Thread kelly . ohair
Changeset: 7e13dbf7e8af Author:ohair Date: 2011-04-06 22:14 -0700 URL: http://hg.openjdk.java.net/jdk7/build/rev/7e13dbf7e8af 7033660: Update copyright year to 2011 on any files changed in 2011 Reviewed-by: dholmes ! Makefile ! make/Defs-internal.gmk ! make/corba-rules.gmk ! make/d

hg: jdk7/build/corba: 7033660: Update copyright year to 2011 on any files changed in 2011

2011-04-06 Thread kelly . ohair
Changeset: 53c6f4ad1e06 Author:ohair Date: 2011-04-06 20:36 -0700 URL: http://hg.openjdk.java.net/jdk7/build/corba/rev/53c6f4ad1e06 7033660: Update copyright year to 2011 on any files changed in 2011 Reviewed-by: dholmes ! src/share/classes/com/sun/corba/se/impl/orbutil/resources/s

Re: Review please: CR: 7033954 / add mapfiles to the launchers and unpack200.

2011-04-06 Thread Kumar Srinivasan
Hi Kelly, I think this looks fine. There is nothing special about the name SUNWprivate_1.1, if you want a different name we can accommodate that. True, but we check for SUNWprivate though for instance, so I guess for now we should keep everything consistent. ex: % grep SUNWprivate * Defs-li

Re: Review please: CR: 7033954 / add mapfiles to the launchers and unpack200.

2011-04-06 Thread Kelly O'Hair
I think this looks fine. There is nothing special about the name SUNWprivate_1.1, if you want a different name we can accommodate that. With JDK8 I'd like to change them all to JDK1.8.0 for the explicitly public externs, and maybe JDK_private_1.8.0 for all the extern symbols only used internal

Re: Need reviewer: Copyright year updates

2011-04-06 Thread David Holmes
Kelly O'Hair said the following on 04/07/11 02:57: I'll reply to myself and see if that email shows up. :^( David, you ok with these changes? Sure. I only took a cursory look. There's very little that can be spotted in such a review as there's no way to know that the file should be updated,

Review please: CR: 7033954 / add mapfiles to the launchers and unpack200.

2011-04-06 Thread Kumar Srinivasan
Hi, This is for adding mapfiles to all launcher based tools as well as unpack200, which eliminates the warning messages for the tools during build. http://cr.openjdk.java.net/~ksrini/7033954/webrev.0/ Thanks Kumar

Re: Avoid certain functions in C/C++

2011-04-06 Thread Mario Torre
Il giorno mer, 06/04/2011 alle 08.50 -0700, Kelly O'Hair ha scritto: > Just an FYI... > > Anyone working with C/C++ should be well aware of the functions we should be > avoiding: > http://hub.opensolaris.org/bin/view/Community+Group+security/funclist > > Microsoft has used the term "banned" an

Re: Need reviewer: Copyright year updates

2011-04-06 Thread Kelly O'Hair
I'll reply to myself and see if that email shows up. :^( David, you ok with these changes? -kto On Apr 6, 2011, at 8:15 AM, Kelly O'Hair wrote: > Correct. I ran this first, to make sure some of the last 2010 changesets > pushed in were covered for 2010: > sh ./make/scripts/update_copyright

Avoid certain functions in C/C++

2011-04-06 Thread Kelly O'Hair
Just an FYI... Anyone working with C/C++ should be well aware of the functions we should be avoiding: http://hub.opensolaris.org/bin/view/Community+Group+security/funclist Microsoft has used the term "banned" and has a much more extensive list: http://msdn.microsoft.com/en-us/library/bb2884

Re: Need reviewer: Turn off warnings for mapfiles in fastdebug builds

2011-04-06 Thread Kelly O'Hair
On Apr 6, 2011, at 6:07 AM, Daniel D. Daugherty wrote: > On 4/5/2011 9:35 PM, Kelly O'Hair wrote: >> >> >> On Apr 5, 2011, at 8:30 PM, Daniel D. Daugherty wrote: >> >>> Looks good to me also. :-) >>> >>> Dan >>> >>> P.S. >>> So the missing "endif" at the end didn't cause any errors? >> >> B

Re: Need reviewer: Turn off warnings for mapfiles in fastdebug builds

2011-04-06 Thread Daniel D. Daugherty
On 4/5/2011 9:35 PM, Kelly O'Hair wrote: On Apr 5, 2011, at 8:30 PM, Daniel D. Daugherty wrote: Looks good to me also. :-) Dan P.S. So the missing "endif" at the end didn't cause any errors? Before I changed the files, they ended with "endif" and no newli

hg: jdk7/build/hotspot: 31 new changesets

2011-04-06 Thread christine . lu
Changeset: 74e790c48cd4 Author:sla Date: 2011-03-28 12:48 +0200 URL: http://hg.openjdk.java.net/jdk7/build/hotspot/rev/74e790c48cd4 7031571: Generate native VS2010 project files Reviewed-by: hosterda, stefank, brutisso ! make/windows/create.bat ! make/windows/makefiles/projectcreat