diff options
author | Коренберг Марк <mark@ideco.ru> | 2012-06-08 14:15:06 (GMT) |
---|---|---|
committer | Thomas Graf <tgraf@redhat.com> | 2012-06-13 11:30:26 (GMT) |
commit | 2bdcde7e8e8bb78b165f093f1a708134f417e557 (patch) | |
tree | 52d56b1b1d1f75373a33c27988ded96e6a62e36e /lib/route/qdisc.c | |
parent | 4f933648622fff2b7fd6ec6c71724da4992c2544 (diff) | |
download | libnl-2bdcde7e8e8bb78b165f093f1a708134f417e557.zip libnl-2bdcde7e8e8bb78b165f093f1a708134f417e557.tar.gz libnl-2bdcde7e8e8bb78b165f093f1a708134f417e557.tar.bz2 |
Fix types-related warnings based on clang diagnostics
1. Fix some places where unsigned value compared < 0
2. Fix obsolete %Z specifier to more portable %z
3. Some erroneous types substitution
4. nl_msec2str() - 64-bit msec is now properly used,
Only safe changes. I mean int <--> uint32_t and signed/unsigned fixes.
Some functinos require size_t argument instead of int, but changes of
signatures of that functions is terrible thing.
Also, I do not pretend for a full list of fixes.
Just to shut up clang -Wall -Wextra
One more thing. ifindex. I don't change that because changes will
be too big for simple fix.
Diffstat (limited to 'lib/route/qdisc.c')
-rw-r--r-- | lib/route/qdisc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/route/qdisc.c b/lib/route/qdisc.c index e5a8aa0..3d618c6 100644 --- a/lib/route/qdisc.c +++ b/lib/route/qdisc.c @@ -275,7 +275,7 @@ int rtnl_qdisc_build_delete_request(struct rtnl_qdisc *qdisc, { struct nl_msg *msg; struct tcmsg tchdr; - int required = TCA_ATTR_IFINDEX | TCA_ATTR_PARENT; + uint32_t required = TCA_ATTR_IFINDEX | TCA_ATTR_PARENT; if ((qdisc->ce_mask & required) != required) { APPBUG("ifindex and parent must be specified"); |