PATCH: IBM AIX patches
Pavel Roskin
proski at gnu.org
Tue Sep 4 23:33:27 UTC 2001
Hi, Pavel!
> > ! /* //#define CPIO_POS(super) (super)->u.cpio.fd */ /* If some time reentrancy should be needed */
>
> You shoud have removed "//", not leave it there. Ditto few more places.
Doesn't matter. This was fixed correctly long ago.
> > STATUS_EOFMARK,
> > ! STATUS_EOF,
> > } ReadStatus;
> > /*
> > * Return 1 for success, 0 if the checksum is bad, EOF on eof,
>
> What's wrong with this? I thought C allowed leaving "," at the end, and
> it is easier to add to list like this.
It's easier, but it doesn't work on AIX. I have to separate reports, so
it must be a problem. It may be a bug in the AIX compiler if the standard
allows that comma, but it's easier to fix it now that to fix it later in
vi over a dumb terminal.
By the way, "gcc -pedantic" would also notice this comma.
--
Regards,
Pavel Roskin
More information about the mc-devel
mailing list