cvs commit: src/sys/net if.c

Ed Maste emaste at FreeBSD.org
Wed Dec 21 21:05:04 PST 2005


emaste      2005-12-22 05:05:00 UTC

  FreeBSD src repository

  Modified files:        (Branch: RELENG_5)
    sys/net              if.c 
  Log:
  MFC r1.235, r1.237, r1.238:
  
    (1.235)
    - Introduce a helper function if_setflag() containing the code common
      to ifpromisc() and if_allmulti() instead of duplicating the code poorly,
      with different bugs.
    - Call ifp->if_ioctl() in a consistent way: always use more compatible C
      syntax and check whether ifp->if_ioctl is not NULL prior to the call.
  
    (1.237)
    Rename equal() macro to sa_equal(), which matches the definitions
    of sa_equal() in other files, and makes it more clear what equal()
    is comparing.
  
    (1.238)
    In multicast routines:
  
    Compare pointers with NULL rather than treating them as booleans.
  
    Compare pointers with NULL rather than 0 to make it more clear
    they are pointers.
  
    Assign pointers value of NULL rather than 0 to make it more clear
    they are pointers.
  
  Approved by:    scottl
  Silence from:   rwatson (mentor)
  
  Revision    Changes    Path
  1.199.2.19  +127 -100  src/sys/net/if.c


More information about the cvs-src mailing list