RFR: 8305538: Avoid redundant HashMap.containsKey call in ModuleDescriptor.Builder

[Andrey Turbanov]

On Tue, 4 Apr 2023 13:57:26 GMT, Alan Bateman <alanb at openjdk.org> wrote:

> If you are changing uses(String) then it should obey `strict`, meaning put this at the beginning of the method
> 
> ```
> if (strict) {
>     requireServiceTypeName(service);
> }
> ```

But now `requireServiceTypeName` is called unconditionally. Is it a bug?

-------------

PR Comment: https://git.openjdk.org/jdk/pull/13288#issuecomment-1496581270