review request (M) 6888954 - arg formatting for assert

Christian Thalinger Christian.Thalinger at Sun.COM
Mon Oct 12 09:02:59 PDT 2009


On Sat, 2009-10-10 at 16:21 -0700, John Coomes wrote:
> 6888954: argument formatting for assert() and friends
> 
> Enable argument formatting for assert(), guarantee() and related
> macros.
> 
> http://cr.openjdk.java.net/~jcoomes/6888954-assert-args/

Except the curly brace on an extra line:

+class FormatBuffer
+{

it looks good.  Thanks for doing that change!

-- Christian



More information about the hotspot-dev mailing list