Change naked constants into pre-processor defines

[edwardhartnett]

In bacio.c we have code like this (note the use of bare constants in return statements):

    /* Check for illegal combinations of options */
    if ((BAOPEN_RONLY & *mode) &&
        ((BAOPEN_WONLY & *mode) || (BAOPEN_WONLY_TRUNC & *mode) || (BAOPEN_WONLY_APPEND & *mode)))
        return 255;

    if ((BAREAD & *mode) && (BAWRITE & *mode)) 
        return 254;

    /* Copy and null terminate the filename. */
    if ((BAOPEN_RONLY & *mode) || (BAOPEN_WONLY & *mode) ||
        (BAOPEN_WONLY_TRUNC & *mode) || (BAOPEN_WONLY_APPEND & *mode) ||
        (BAOPEN_RW & *mode))
    {
        if (!(realname = (char *) malloc((namelen + 1) * sizeof(char))))
            return 253;

Of course these should be pre-processor defines, like well-behaved C code...

[edwardhartnett]

These changes got lost in a merge...