Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle cases of using class with generics for @BindingAdapter in binding-adapter-bridge #17

Merged

Conversation

arunkumar9t2
Copy link
Contributor

@arunkumar9t2 arunkumar9t2 commented Jan 25, 2022

Support cases where @BindingAdapter is statically declared in a class that uses generic in its type params. This change fixes code generation to not generate generic information in generated code for those classes since static access is invalid in Java/Kotlin.

-    CompanionTest<T>.companionUpdateNode(node, value);
+   CompanionTest.companionUpdateNode(node, value);

Fixes #16

@arunkumar9t2 arunkumar9t2 self-assigned this Jan 25, 2022
@arunkumar9t2 arunkumar9t2 added the bug Something isn't working label Jan 25, 2022
…nding-adapter-bridge

Support cases where `@BindingAdapter` is statically declared in a class that uses generic in its type params. This change fixes code generation to not generate generic information in generated code for those classes since static access is invalid in Java/Kotlin.

Fixes #16
@arunkumar9t2 arunkumar9t2 force-pushed the fix/kotlin-binding-adapters-with-parent-class-generic branch from d6f8409 to 4db6d07 Compare January 25, 2022 19:08
@arunkumar9t2 arunkumar9t2 merged commit ad92dbd into master Jan 26, 2022
@arunkumar9t2 arunkumar9t2 deleted the fix/kotlin-binding-adapters-with-parent-class-generic branch January 26, 2022 05:40
akshay-grabtaxi pushed a commit that referenced this pull request Jun 28, 2023
…nding-adapter-bridge (#17)

Support cases where `@BindingAdapter` is statically declared in a class that uses generic in its type params. This change fixes code generation to not generate generic information in generated code for those classes since static access is invalid in Java/Kotlin.

Fixes #16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue using Kotlin BindingAdapters
3 participants