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

<html>

  <head>

    <meta content="text/html; charset=ISO-8859-1"

      http-equiv="Content-Type">

  </head>

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

    <font face="DejaVu Sans">Sounds good.<br>

      <br>

      Thanks,<br>

      Justin<br>

      <br>

    </font>On 05/09/2012 12:34 AM, Thomas Graf wrote:

    <blockquote cite="mid:20120509063457.GK21089@canuck.infradead.org"

      type="cite">

      <pre wrap="">

On Tue, May 08, 2012 at 04:17:58PM -0600, Justin Mayfield wrote:

</pre>

      <blockquote type="cite">

        <pre wrap="">I have a question/comment about a minor API inconsistency.  I have a

routine that takes function pointers from the following functions:

   rtnl_rule_set_src

   rtnl_rule_set_dst

   rtnl_route_set_src

   rtnl_route_set_dst

   rtnl_route_set_pref_src

   rtnl_route_nh_set_gateway

All of these except for rtnl_route_nh_set_gateway() return an int.

I can easily wrap this but it seemed worth mentioning that of all

the nl_addr "setters" this was the only one to not return an int.

</pre>

      </blockquote>

      <pre wrap="">

I agree that it would make sense to have rtnl_route_nh_set_gateway()

return an int as well. It's simply a bug. I don't want to fix it at

this point though, as it would break ABI with only little to gain.

I have put in a FIXME so we can change this when we break API/ABI for

real next time (libnl 4.x).

Good enough for you?

~Thomas

!SIG:4faa101a171621322317785!

</pre>

    </blockquote>

  </body>

</html>