> On 12 Nov 2015, at 15:24, Roger Riggs <roger.ri...@oracle.com> wrote: > > Hi Paul, > > Thanks. >
One more thing i forgot to mention, if there is ever a chance that ProcessBuilder is changed from final to non-final we should List<? extends ProcessBuilder>. > One followup comment comparing varargs to List<T>. > > If there is any concern about concurrency, the callee still has to make a > copy of the incoming > List/array before using/check it to get a consistent view. Only the caller > can rely on the immutability > of the argument (if it created it); not the callee. Now if it were a > concrete final type... > We still have to watch out for that. > Yes. Paul.