7016704: TEST_BUG: java/nio/file/Files/walk_file_tree.sh fails with new version of find (lnx)

[Rémi Forax]

On 02/14/2011 03:13 PM, Alan Bateman wrote:
> Rémi Forax wrote:
>>
>> Alan,
>> I don't understand
>>
>>   assert exc != null;
>>
>> Unlike in postVisitDirectory(), in visitFileFailed() the exception 
>> should be never null.
>> I think the assert should be move before the if.
>>
> The visitFileFailed method is always called with an IOException and so 
> exc is never null. We can get rid of assert, it's not needed - is that 
> what you mean?
>
> -Alan.

Get rid of it or put it on top of visitFileFailed().
The question was more, why put that assert only in the else branch ?

Rémi