RE: RFR(s): 8152617 add missing wildcards to Optional or()andflatMap()

2016-10-12 Thread timo.kinnunen
Hi, I know of in/inout/out parameters from GLSL and, with most all conversions in GLSL being explicit, they are the easiest thing ever to reason about. In GLSL a 2D vector and a float array of 2 can both be accessed using array indexing, but nobody cares if one should be a subtype of another o

RE: RFR(s): 8152617 add missing wildcards to Optional or() andflatMap()

2016-10-12 Thread timo.kinnunen
Hi, I’m going to challenge the consensus a little bit. First, Rémi's example can be simplified to this one method which fails to compile with the same error message[0]: private static Optional simple1( Optional o, Function> f ) {return o.flatMap(f);} Second, although not in the webrev, Opt

RE: Make iterators cloneable?

2016-09-12 Thread timo.kinnunen
No, It breaks on Fi (or Fum) in the inner loop before it gets the chance to output Fi-Fum (or Fum-Fi) if Fi (respectively Fum) comes first in iteration order. -- Have a nice day, Timo Sent from Mail for Windows 10 From: Dave Brosius

RE: A new helper method Arrays.asArray

2016-06-14 Thread timo.kinnunen
The same issue is present even without any varargs or generics present, when using plain old arrays: String[] list1 = new String[1]; list1[0] = "hello"; Integer[] list2 = new Integer[1]; list2[0] = 1; String[][] array

RE: A new helper method Arrays.asArray

2016-06-14 Thread timo.kinnunen
I think @SafeVarargs should be OK here since the array containing the varargs already exists before the method starts executing and so the annotation amounts to asserting that the passed-in vararg-array is just as safe during the execution of the method as it is after the method execution comple

A new helper method Arrays.asArray

2016-06-14 Thread timo.kinnunen
Hi, I have found that many times I need to write this simple helper method: public static @SafeVarargs T[] asArray(T… ts) { return ts; } I usually need this when I have several implementations I’m comparing and I want to change the code for observing one of them to observing two or

RE: Fwd: Files.walk() is unusable because of AccessDeniedException

2016-05-26 Thread timo.kinnunen
Let’s not forget that in the use case where there is an installed security manager, the API clearly states how those access checks are supported. The results of access checks are communicated by the security manager using unchecked exceptions. As a result the Files::walk implementation has to c

RE: Fwd: Files.walk() is unusable because of AccessDeniedException

2016-05-24 Thread timo.kinnunen
Hi, The API documentation makes the point that if there is a security manager installed which denies access to some entries amongst the visited files and directories then those entries are silently skipped, their presence not being disclosed by the walk terminating early due to an access denie