On Sat, 11 Jul 2026 17:01:09 GMT, Petr Štechmüller <[email protected]> wrote:

>> What I meant is only the condition `!strsMap.containsKey(propName)`, was 
>> just wondering because it is only done here and not for the other types
>
> By "all other types" you mean `Set` and `List`? They are covered by 
> `Collection` check above.

What I mean is, we have the following conditions:
- `Collection.class.isAssignableFrom(retType) && argType.length == 0`
- `Map.class.isAssignableFrom(retType) && argType.length == 0 && 
!strsMap.containsKey(propName)`
- `ObservableArray.class.isAssignableFrom(retType) && argType.length == 0`

And I was wondering why only for the `Map` if condition we have the additional 
`!strsMap.containsKey(propName)` check

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2167#discussion_r3564658109

Reply via email to