RFR JDK-8039525: HashTable lookup API needs refactoring

[John Rose]

Three comments:

1. Yay!  I have a soft spot for this kind of change.

2. You apparently added a blank-carrying line in symbolTable.hpp.  jcheck will stop you from committing it.

3. Please get rid of "probe*" while you are at it, or at least pick a name which fits in your new scheme.  IIRC it was an incursion from a non-runtime programmer.

Reviewed.

— John

On Apr 24, 2014, at 1:59 PM, Gerard Ziemski <gerard.ziemski at oracle.com> wrote:

> hi all,
> 
> Please review this simple enhancement, which renames symbolTable::lookup API using names that better reflect the method purpose, ie: lookup_only, lookup_and_add, lookup_only_and_hash, lookup_and_add_unicode and lookup_only_and_hash_unicode.
> 
> No new or removed code.
> 
> Webrev: http://cr.openjdk.java.net/~gziemski/8039525_rev0
> Bug: https://bugs.openjdk.java.net/browse/JDK-8039525
> 
> Tested with hotspot jtreg, vm.quicklooks and JPRT. Locally builds on Mac OS X and Linux.
> 
> Thank you.