RFR: JDK-8143255: Remove debug logging from SymbolTable::unlink() and SymbolTable::possibly_parallel_unlink() (original) (raw)
Bengt Rutisson bengt.rutisson at oracle.com
Thu Nov 19 09:04:06 UTC 2015
- Previous message: RFR: JDK-8143255: Remove debug logging from SymbolTable::unlink() and SymbolTable::possibly_parallel_unlink()
- Next message: RFR: JDK-8143255: Remove debug logging from SymbolTable::unlink() and SymbolTable::possibly_parallel_unlink()
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 2015-11-19 10:00, Thomas Schatzl wrote:
Hi Bengt,
On Thu, 2015-11-19 at 09:24 +0100, Bengt Rutisson wrote: Hi Thomas,
Thanks for looking at this! On 2015-11-18 21:36, Thomas Schatzl wrote: Hi Bengt,
On Wed, 2015-11-18 at 20:47 +0100, Bengt Rutisson wrote: Hi everyone,
Could I have a couple of reviews for this patch to remove some debugging code? http://cr.openjdk.java.net/~brutisso/8143255/webrev.00/ https://bugs.openjdk.java.net/browse/JDK-8143255 afaik the memorytotal parameter to the various methods is only every consumed by the now removed printout. Can you please remove that too? Good point. Here's an updated webrev: http://cr.openjdk.java.net/~brutisso/8143255/webrev.01/ looks good.
Thanks, Thomas! Bengt
Thomas
- Previous message: RFR: JDK-8143255: Remove debug logging from SymbolTable::unlink() and SymbolTable::possibly_parallel_unlink()
- Next message: RFR: JDK-8143255: Remove debug logging from SymbolTable::unlink() and SymbolTable::possibly_parallel_unlink()
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]