diff options
author | J. Bruce Fields <bfields@fieldses.org> | 2006-11-14 16:54:36 -0500 |
---|---|---|
committer | J. Bruce Fields <bfields@citi.umich.edu> | 2007-10-09 18:32:45 -0400 |
commit | 526985b9dd6ef7716b87f5fe6f0e2438ea3a89c7 (patch) | |
tree | e22c895d58413a2b784b7e91a9c6d73e724a1838 /fs | |
parent | b842e240f27678aa5d71611cddc8d17a93fb0caf (diff) |
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')
-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 | |||
819 | lock_kernel(); | 819 | lock_kernel(); |
820 | if (request->fl_type != F_UNLCK) { | 820 | if (request->fl_type != F_UNLCK) { |
821 | for_each_lock(inode, before) { | 821 | for_each_lock(inode, before) { |
822 | struct file_lock *fl = *before; | 822 | fl = *before; |
823 | if (!IS_POSIX(fl)) | 823 | if (!IS_POSIX(fl)) |
824 | continue; | 824 | continue; |
825 | if (!posix_locks_conflict(request, fl)) | 825 | if (!posix_locks_conflict(request, fl)) |