RFR: 8179181: Cleanup BitMap search API

[coleen.phillimore at oracle.com]

Looks good.   Second review.

On 4/23/17 5:49 AM, Kim Barrett wrote:
>> On Apr 22, 2017, at 10:47 PM, David Holmes <david.holmes at oracle.com> wrote:
>>
>> Hi Kim,
>>
>> On 23/04/2017 3:03 AM, Kim Barrett wrote:
>>> Please review this small simplification of the BitMap search API. The
>>> change consists of dropping the _inline suffix from some function
>>> names, and removing the now redundant (and also declared inline)
>>> forwarding wrappers.
>> That was one seriously confused API! Presumably the intent was to have inlined and non-lined variants, but as you note everything was inline anyway. :(
> I particularly liked the "Non-inline versionsof the above.” comment, and yet, there they are in the .inline.hpp…

Seriously, thanks for cleaning this up.
Coleen
>
>> Cleanup looks good.
> Thanks.
>
>> Thanks,
>> David
>>
>>> CR:
>>> https://bugs.openjdk.java.net/browse/JDK-8179181
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~kbarrett/8179181/hotspot.00/
>>>
>>> Testing:
>>> JPRT
>