]> granicus.if.org Git - mutt/commit
projects / mutt / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 2d88922) | patch
Remove extra lstats() in safe_rename() upon link success.
authorKevin McCarthy <kevin@8t8.us>
Wed, 29 Aug 2018 20:00:14 +0000 (13:00 -0700)
committerKevin McCarthy <kevin@8t8.us>
Wed, 29 Aug 2018 20:00:14 +0000 (13:00 -0700)
commit6216a753e0207ff2be9b994f0cd82123037cf1bf
tree9a3e09b4b3f2bfcc36fd7dbc2cfe3bc9c583ff8ftree | snapshot
parent2d88922756c9c72d5530308383227851c21160fbcommit | diff
Remove extra lstats() in safe_rename() upon link success.

I originally kept the lstat()'s in order to make as few changes as
possible to the function.  However, Derek Martin argues correctly that
if we are trusting the link() 0-retval, the lstats are unnecessary and
in fact can cause a performance impact over NFS.
lib.c diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.