Skip to contentSkip to conversation

__FUNCTION__ string concatenation

PATCH In review [email protected] 1 participant 2 messages Sep 23, 2004 Networking drivers (wireless)

lore
Share & tools
lore
#0 lore raw

PATCH 2/3 __FUNCTION__ string concatenation

I've replaced the __FUNCTION__ string concatenation with the %s placeholder and a printf parameter in drivers/net/wireless/prism65/islpci_mgt.h, as suggested in the TODO list.

I don't have the hardware to do a run-time check. It should not pose any problems though.

# This is a BitKeeper generated diff -Nru style patch. #
# ChangeSet
# 2004/07/04 22:08:37+02:00 [email protected]
# __FUNCTION__ string concatenation is deprecated
#

# drivers/net/wireless/prism54/islpci_mgt.h
#   2004/07/03 17:18:20+02:00 [email protected] +1 -1
#   __FUNCTION__ string concatenation is deprecated
# 

---

 linux-2.6.9-rc2-bk7-max/drivers/net/wireless/prism54/islpci_mgt.h |    2 +-
 1 files changed, 1 insertion(+), 1 deletion(-)
drivers/net/wireless/prism54/islpci_mgt.h renamed from drivers/net/wireless/prism54/islpci_mgt.h~printk-net_wireless_prism54_islpci_mgt.h +1 -1
diff -puN drivers/net/wireless/prism54/islpci_mgt.h~printk-net_wireless_prism54_islpci_mgt.h drivers/net/wireless/prism54/islpci_mgt.h
@@ -31,7 +31,7 @@
 #define K_DEBUG(f, m, args...) do { if(f & m) printk(KERN_DEBUG args); } while(0)
 #define DEBUG(f, args...) K_DEBUG(f, pc_debug, args)
 
-#define TRACE(devname)   K_DEBUG(SHOW_TRACING, VERBOSE, "%s:  -> " __FUNCTION__ "()\n", devname)
+#define TRACE(devname)   K_DEBUG(SHOW_TRACING, VERBOSE, "%s:  -> %s()\n", devname, __FUNCTION__)
 
 extern int pc_debug;
 #define init_wds 0	/* help compiler optimize away dead code */

_

#1 lore raw
Quoted 10 lines · On Thu, 23 Sep 2004, [email protected] wrote:

I've replaced the __FUNCTION__ string concatenation with the %s placeholder and a printf parameter in drivers/net/wireless/prism65/islpci_mgt.h, as suggested in the TODO list.

I don't have the hardware to do a run-time check. It should not pose any problems though.

argh that one is wrong please forget that single one.

thanks maks