In message <[EMAIL PROTECTED]>,"Marc Dionne" writes: >> if (fp->f_flags | O_RDONLY) /* readers dont flush */ > >Not sure if it has any effect, but this doesn't look right. Perhaps >the intention was to use &, but that wouldn't work either since >O_RDONLY happens to be 0. > >Typically it would look more like this: > if ((fp->f_flags & O_ACCMODE) == O_RDONLY)
whoops. that is my brain damage. i told derrick to do that, and i confuse | and & all the time. even so, O_RDONLY is 0x0, so your version is certainly correct. _______________________________________________ OpenAFS-devel mailing list [email protected] https://lists.openafs.org/mailman/listinfo/openafs-devel
