From e408b9856a7ceb93ad8d683d0070e3b2908e9341 Mon Sep 17 00:00:00 2001
From: Alexey Kudravtsev <cdr@intellij.com>
Date: Fri, 5 Oct 2018 17:55:33 +0300
Subject: [PATCH] Use bounded wildcards in statics/constructors/final methods
 in internal API to make it more conventional, flexible and documented

---
 platform/lang-api/src/com/intellij/semantic/SemRegistrar.java | 4 ++--
 .../lang-impl/src/com/intellij/semantic/SemServiceImpl.java   | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/platform/lang-api/src/com/intellij/semantic/SemRegistrar.java b/platform/lang-api/src/com/intellij/semantic/SemRegistrar.java
index 46349ef938e5..8137f0e8be3f 100644
--- a/platform/lang-api/src/com/intellij/semantic/SemRegistrar.java
+++ b/platform/lang-api/src/com/intellij/semantic/SemRegistrar.java
@@ -27,8 +27,8 @@ import java.util.Collection;
  */
 public interface SemRegistrar {
 
-  <T extends SemElement, V extends PsiElement> void registerSemElementProvider(SemKey<T> key, ElementPattern<? extends V> place, NullableFunction<V, T> provider);
+  <T extends SemElement, V extends PsiElement> void registerSemElementProvider(SemKey<T> key, ElementPattern<? extends V> place, NullableFunction<? super V, ? extends T> provider);
 
-  <T extends SemElement, V extends PsiElement> void registerRepeatableSemElementProvider(SemKey<T> key, ElementPattern<? extends V> place, NullableFunction<V, Collection<T>> provider);
+  <T extends SemElement, V extends PsiElement> void registerRepeatableSemElementProvider(SemKey<T> key, ElementPattern<? extends V> place, NullableFunction<? super V, ? extends Collection<T>> provider);
 
 }
diff --git a/platform/lang-impl/src/com/intellij/semantic/SemServiceImpl.java b/platform/lang-impl/src/com/intellij/semantic/SemServiceImpl.java
index 9b64403b89fd..1c49649a82dd 100644
--- a/platform/lang-impl/src/com/intellij/semantic/SemServiceImpl.java
+++ b/platform/lang-impl/src/com/intellij/semantic/SemServiceImpl.java
@@ -87,7 +87,7 @@ public class SemServiceImpl extends SemService{
       @Override
       public <T extends SemElement, V extends PsiElement> void registerSemElementProvider(SemKey<T> key,
                                                                                           final ElementPattern<? extends V> place,
-                                                                                          final NullableFunction<V, T> provider) {
+                                                                                          final NullableFunction<? super V, ? extends T> provider) {
         map.putValue(key, element -> {
           if (place.accepts(element)) {
             return Collections.singleton(provider.fun((V)element));
@@ -99,7 +99,7 @@ public class SemServiceImpl extends SemService{
       @Override
       public <T extends SemElement, V extends PsiElement> void registerRepeatableSemElementProvider(SemKey<T> key,
                                                                                                     ElementPattern<? extends V> place,
-                                                                                                    NullableFunction<V, Collection<T>> provider) {
+                                                                                                    NullableFunction<? super V, ? extends Collection<T>> provider) {
         map.putValue(key, element -> {
           if (place.accepts(element)) {
             return provider.fun((V)element);