summaryrefslogtreecommitdiffstats
path: root/COPYING
diff options
context:
space:
mode:
authorJulia Lawall <Julia.Lawall@inria.fr>2020-07-26 12:58:31 +0200
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>2020-09-01 14:12:36 +0200
commit3c9c3902e325b03e1c3b5f95f7274086487498bd (patch)
tree15adb477dcb63f8b37716a6ae1234fe134334636 /COPYING
parenta357a7840702d24a7d55d0f950af35b1be1ee005 (diff)
downloadlinux-3c9c3902e325b03e1c3b5f95f7274086487498bd.tar.bz2
tm6000: drop unnecessary list_empty
list_for_each_entry is able to handle an empty list. The only effect of avoiding the loop is not initializing the index variable. Drop list_empty tests in cases where these variables are not used. Note that list_for_each_entry is defined in terms of list_first_entry, which indicates that it should not be used on an empty list. But in list_for_each_entry, the element obtained by list_first_entry is not really accessed, only the address of its list_head field is compared to the address of the list head, so the list_first_entry is safe. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) <smpl> @@ expression x,e; iterator name list_for_each_entry; statement S; identifier i; @@ -if (!(list_empty(x))) { list_for_each_entry(i,x,...) S - } ... when != i ? i = e </smpl> Signed-off-by: Julia Lawall <Julia.Lawall@inria.fr> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Diffstat (limited to 'COPYING')
0 files changed, 0 insertions, 0 deletions