RFR: 8238761: Asynchronous handshakes [v5]
    Robbin Ehn 
    rehn at openjdk.java.net
       
    Wed Sep 23 10:52:27 UTC 2020
    
    
  
On Wed, 23 Sep 2020 04:06:46 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Robbin Ehn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update after Coleen
>
> src/hotspot/share/utilities/filterQueue.hpp line 56:
> 
>> 54:   }
>> 55:
>> 56:   // MT-safe
> 
> Not sure where our previous discussion is on this but these posix-style MT-safe labels don't really halp for this kind
> of abstract data type API. Please briefly explain the thread-safety properties of add and pop.
Added comment
> src/hotspot/share/utilities/filterQueue.hpp line 57:
> 
>> 55:
>> 56:   // MT-safe
>> 57:   void add(E data);
> 
> It would be more regular naming to use add/remove or push/pop rather than add/pop.
Change to push.
-------------
PR: https://git.openjdk.java.net/jdk/pull/151
    
    
More information about the serviceability-dev
mailing list