diff options
| author | J. Bruce Fields | 2006-11-14 22:54:36 +0100 |
|---|---|---|
| committer | J. Bruce Fields | 2007-10-10 00:32:45 +0200 |
| commit | 526985b9dd6ef7716b87f5fe6f0e2438ea3a89c7 (patch) | |
| tree | e22c895d58413a2b784b7e91a9c6d73e724a1838 /fs/locks.c | |
| parent | locks: reverse order of posix_locks_conflict() arguments (diff) | |
| download | kernel-qcow2-linux-526985b9dd6ef7716b87f5fe6f0e2438ea3a89c7.tar.gz kernel-qcow2-linux-526985b9dd6ef7716b87f5fe6f0e2438ea3a89c7.tar.xz kernel-qcow2-linux-526985b9dd6ef7716b87f5fe6f0e2438ea3a89c7.zip | |
locks: kill redundant local variable
There's no need for another variable local to this loop; we can use the
variable (of the same name!) already declared at the top of the function,
and not used till later (at which point it's initialized, so this is safe).
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Diffstat (limited to 'fs/locks.c')
| -rw-r--r-- | fs/locks.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/locks.c b/fs/locks.c index 51bae6227c25..efe1affe6bed 100644 --- a/fs/locks.c +++ b/fs/locks.c @@ -819,7 +819,7 @@ static int __posix_lock_file(struct inode *inode, struct file_lock *request, str lock_kernel(); if (request->fl_type != F_UNLCK) { for_each_lock(inode, before) { - struct file_lock *fl = *before; + fl = *before; if (!IS_POSIX(fl)) continue; if (!posix_locks_conflict(request, fl)) |