Re: Java_java_io_File_setReadOnlyInternal bug ?

[Aaron M. Renn]

>It should make file readonly. Current implementation uses
>
>newmode = newmode ^ (S_IWRITE|S_IWGRP|S_IWOTH);
>
>I think that it should be
>
>newmode = newmode & (~(S_IWRITE|S_IWGRP|S_IWOTH));
>
>unless I'm mistaken in understanding chmod functionality.

Obviously I didn't test this function.  The fix is checked in. Thanks.

BTW: I haven't forgotten about the encoders, I just haven't done anything
yet.

-- 
Aaron M. Renn ([EMAIL PROTECTED]) http://www.urbanophile.com/arenn/