RFR: JDK-8143255: Remove debug logging from SymbolTable::unlink() and SymbolTable::possibly_parallel_unlink() (original) (raw)
Thomas Schatzl thomas.schatzl at oracle.com
Thu Nov 19 09:00:30 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 ]
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.
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 ]