Request for review (S): JDK-8011009: Use do-while(0) instead of while(0) in EC_TRACE and RC_TRACE* macros (original) (raw)
serguei.spitsyn at oracle.com serguei.spitsyn at oracle.com
Thu Mar 28 15:24:56 PDT 2013
- Previous message: Request for review (S): JDK-8011009: Use do-while(0) instead of while(0) in EC_TRACE and RC_TRACE* macros
- Next message: Request for review (S): JDK-8011009: Use do-while(0) instead of while(0) in EC_TRACE and RC_TRACE* macros
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hi Krystal,
The fix is good.
Thanks, Serguei
On 3/28/13 3:11 PM, Krystal Mo wrote:
Hi all,
Could I have a review for this small patch, please? Webrev: http://cr.openjdk.java.net/~kmo/8011009/webrev.00 CR: http://bugs.sun.com/bugdatabase/viewbug.do?bugid=8011009 This patch improves the ECTRACE and RCTRACE* family of macros to be less error prone. They used to end with while(0) to make the macros statement-like, but a better way to do this is do {...} while(0). Testing with JPRT, will send an update email when it's done. P.S. I'm not sure where I should send this email for review, should it be hotspot-runtime or serviceability? Thanks, Kris
- Previous message: Request for review (S): JDK-8011009: Use do-while(0) instead of while(0) in EC_TRACE and RC_TRACE* macros
- Next message: Request for review (S): JDK-8011009: Use do-while(0) instead of while(0) in EC_TRACE and RC_TRACE* macros
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]