[GROOVY-7976] Sort methods that accept a comparator should accept Comparator<? super T> (fix DGM signatures)

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.4.7
• Fix Version/s: 2.4.8
• Component/s: groovy-jdk
• Assignee: Paul King
• Reporter: Mauro Molinari
• Created: 26/Oct/16 14:51
• Updated: 01/Feb/17 23:19
• Resolved: 07/Nov/16 00:51

Description

Methods like org.codehaus.groovy.runtime.DefaultGroovyMethods.sort(Iterable<T>, boolean, Comparator<T>) and all the others alike should be changed to accept a Comparator<? super T>, otherwise if you use @CompileStatic you'll get compiler errors as soon as you try to sort an Iterable/Collection using a comparator that accept superclass instances.

Issue Links

• is cloned by: GROOVY-7992 Sort methods that accept a comparator should accept Comparator<? super T> (fix type checker ability to cope with super in DGM methods)
• links to: GitHub Pull Request #454