diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2015-09-16 18:35:09 +0200 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2015-09-25 12:04:41 +0200 |
| commit | 5e43efb29ae877da131e6c1a4761cd7f4eec5a16 (patch) | |
| tree | de7e8f1fe58b7310c09d4a6d97c466ec257c13cf /scripts | |
| parent | fe8545386726a2b1f8af409bcd5ea3d33218af54 (diff) | |
| download | qemu-5e43efb29ae877da131e6c1a4761cd7f4eec5a16.tar.gz qemu-5e43efb29ae877da131e6c1a4761cd7f4eec5a16.tar.bz2 qemu-5e43efb29ae877da131e6c1a4761cd7f4eec5a16.zip | |
checkpatch: do not recommend qemu_strtok over strtok
If anything it should recommend strtok_r!
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/checkpatch.pl | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 574334b985..b6d71eae17 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 |