I discovered while wanting to use the second type parameter of net.minecraft.state.State<O,S> in a injected interface, that the second check I made in #1050 was not properly working. In fact, only the first type parameter of the target class was properly read, meaning that if an injected interface wanted to use another type parameter than the first one the check would fail.
This was entirely my fault, I wrongly used SignatureVisitor#visitEnd as I thought this one would be called once SignatureReader would finish to read the signature of the target class.
I also added a test case clearly showing that the issue was solved in the pull request.
I discovered while wanting to use the second type parameter of
net.minecraft.state.State<O,S>in a injected interface, that the second check I made in #1050 was not properly working. In fact, only the first type parameter of the target class was properly read, meaning that if an injected interface wanted to use another type parameter than the first one the check would fail.This was entirely my fault, I wrongly used SignatureVisitor#visitEnd as I thought this one would be called once SignatureReader would finish to read the signature of the target class.
I also added a test case clearly showing that the issue was solved in the pull request.