: * Note: the "volatile" below does not REQUIRE that the argument be
: * volatile, but rather ony says that it is OK to use a volatile * i
: * there. Same for the const. I know a const volatile sounds strange
: * but it only indicates that either is acceptable.
: */
:void    bcopy __P((volatile const void *from, volatile void *to, size_t
:NOW we can get rid of lots of UGLY casting tricks here and there that
:have been used to try UNVOLATILE things so thay they can use bcopy.
:I suggest a similar addition to a few other standard operations.
:Hopefully the BUS-space stuff should be used eventually
:but I'm not going to rewrite if_ie.c..... are you?
:(I've also seem this used in the IPV6 code but I think
:thye've been (yukky) cast'ed by now.

    I really don't like this.  I mean, it will work.... but I really don't
    like this.  Volatile in C will effectively prevent optimization of
    read and write accesses to the data the pointer is pointing at.  Of
    course, this is bcopy(), which we implement in assembly, so again it will
    work for bcopy().  But it sets a very bad coding precedent.

    I would keep the casts.  It reminds us that there is something funny
    going on.

                                        Matthew Dillon 
                                        <[EMAIL PROTECTED]>

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to