From: Paolo Bonzini Date: Wed, 16 Sep 2015 16:35:09 +0000 (+0200) Subject: checkpatch: do not recommend qemu_strtok over strtok X-Git-Tag: TizenStudio_2.0_p2.3.2~120^2~1^2~161^2~47 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5e43efb29ae877da131e6c1a4761cd7f4eec5a16;p=sdk%2Femulator%2Fqemu.git checkpatch: do not recommend qemu_strtok over strtok If anything it should recommend strtok_r! Signed-off-by: Paolo Bonzini --- diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 574334b..b6d71ea 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2463,8 +2463,8 @@ sub process { WARN("__func__ should be used instead of gcc specific __FUNCTION__\n" . $herecurr); } -# recommend qemu_strto* over strto* - if ($line =~ /\b(strto.*?)\s*\(/) { +# recommend qemu_strto* over strto* for numeric conversions + if ($line =~ /\b(strto[^k].*?)\s*\(/) { WARN("consider using qemu_$1 in preference to $1\n" . $herecurr); } # check for module_init(), use category-specific init macros explicitly please