On 17/04/2013 21:00, Chris Hegarty wrote: > Looks fine to me Alan. > > Trivially, the change to the example in the javadoc for > BasicFileAttributeView. Should this be setTimes(null, time, *null*) ?? Thanks Chris, I noticed that too after I generated the webrev. I'll fix it before I push. -Alan.