On 2016/01/25 22:52, Michael McConville wrote:
> fddi_bitswap is only used once, and it just adds a layer of indirection
> to its preprocessor condition.
Oh yuk. This is bogus anyway, and there's no good way to handle it.
We dropped support for FDDI interfaces so it only affect decodes of
pcap files, and who knows where they were created?
> +#if defined(__alpha) || defined(__NetBSD__) || defined(__OpenBSD__)
> + memcpy(fdst, (char *)fddip->fddi_dhost, 6);
> + memcpy(fsrc, (char *)fddip->fddi_shost, 6);
> +#else
> + /*
> + * bit-swap the fddi addresses (isn't the IEEE standards
> + * process wonderful!) then convert them to names.
> + */
> + for (i = 0; i < 6; ++i)
> + fdst[i] = fddi_bit_swap[fddip->fddi_dhost[i]];
> + for (i = 0; i < 6; ++i)
> + fsrc[i] = fddi_bit_swap[fddip->fddi_shost[i]];
> +#endif
> }
>
> /*
>
#if 1
...
#else
/* You might need this depending on where the pcap file was created */
...
#endif
? *shrug*