Github user kinow commented on a diff in the pull request:

    https://github.com/apache/jena/pull/463#discussion_r210856558
  
    --- Diff: jena-base/src/main/java/org/apache/jena/atlas/lib/ListUtils.java 
---
    @@ -34,49 +34,40 @@
     {
         private ListUtils() {}
     
    -    public static <T>
    -    List<T> unique(List<T> list)
    -    {
    -                   return toList(list.stream().distinct()) ;
    +    public static <T> List<T> unique(List<T> list) {
    +        return toList(list.stream().distinct());
         }
    -    
    -    public static
    -    List<Integer> asList(int... values)
    -    {
    -        List<Integer> x = new ArrayList<>(values.length) ;
    +
    +    public static List<Integer> asList(int...values) {
    --- End diff --
    
    Couldn't we replace this method by something like `Arrays.asList(1, 2, 3);` 
? Or maybe deprecate it?


---

Reply via email to