On Tue, 2014-03-18 at 22:56 -0600, Jean Sacren wrote: > Stating it is a "fix" is not a false statement at all. !! falsely > changes the value to be int TWICE for nothing! Are you still not > convinced?
Not so fast Jean. There's no logic change. Changing the type to bool does exactly the same thing and doesn't change the generated object code at all. $ diff net/ieee802154/nl-phy.lst.new net/ieee802154/nl-phy.lst.old 2289d2288 < static int phy_set_lbt(struct wpan_phy *phy, struct genl_info *info) 2291c2290 < u8 on = !!nla_get_u8(info->attrs[IEEE802154_ATTR_LBT_ENABLED]); --- > bool on; 2293a2293 > on = nla_get_u8(info->attrs[IEEE802154_ATTR_LBT_ENABLED]); 2296,2298d2295 < return 0; < } < 2301,2303c2298 < u8 on = !!nla_get_u8(info->attrs[IEEE802154_ATTR_LBT_ENABLED]); < 1096: 80 78 04 00 cmpb $0x0,0x4(%rax) < 109a: 41 0f 95 c5 setne %r13b --- > bool on; 2305a2301,2303 > on = nla_get_u8(info->attrs[IEEE802154_ATTR_LBT_ENABLED]); > 1096: 80 78 04 00 cmpb $0x0,0x4(%rax) > 109a: 41 0f 95 c5 setne %r13b 2322c2320 < --- ------------------------------------------------------------------------------ Learn Graph Databases - Download FREE O'Reilly Book "Graph Databases" is the definitive new guide to graph databases and their applications. Written by three acclaimed leaders in the field, this first edition is now available. Download your free book today! http://p.sf.net/sfu/13534_NeoTech _______________________________________________ Linux-zigbee-devel mailing list Linux-zigbee-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linux-zigbee-devel