
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

  <head>


    <meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">

  </head>

  <body bgcolor="#ffffff" text="#000000">

    <font face="DejaVu Sans">I noticed that many of the

      rtnl_link_vlan_XYZ calls use the macro "IS_VLAN_LINK_ASSERT". 

      Often this is in a function that claims to return < 0 if there

      is an error.  This is only true if the application is compiled

      with NDEBUG because APPBUG does an assert() and thus abort().<br>

      <br>

      I can guard my code to prevent these API calls from being made on

      the wrong link type, but this feels redundant if the

      rtnl_link_vlan_XYZ calls are doing the same check too.  I'd prefer

      that the IS_VLAN_LINK_ASSERT macro be a _CHECK and simply return a

      negative error code which I can parse, as I do for much of the

      other link API.<br>

      <br>

      I can submit a patch to this effect if deemed worthy.<br>

    </font>

  </body>

</html>