diff options
author | Nicholas Clark <nick@ccl4.org> | 2008-03-13 13:01:44 +0000 |
---|---|---|
committer | Nicholas Clark <nick@ccl4.org> | 2008-03-13 13:01:44 +0000 |
commit | 5ca745d26426e8004b3f32338cdb662e30799d8d (patch) | |
tree | a8dd52e9f9e759227f67df8489a67d83b061bc04 /perlio.c | |
parent | 81d4a902ecbc1bf9a78b3df432e698439bf35048 (diff) | |
download | perl-5ca745d26426e8004b3f32338cdb662e30799d8d.tar.gz |
Correct two spelling mistakes, and reformat some comments.
p4raw-id: //depot/perl@33504
Diffstat (limited to 'perlio.c')
-rw-r--r-- | perlio.c | 16 |
1 files changed, 8 insertions, 8 deletions
@@ -3159,20 +3159,20 @@ PerlIOStdio_close(pTHX_ PerlIO *f) if (!invalidate) { #ifdef USE_ITHREADS MUTEX_LOCK(&PL_perlio_mutex); - /* Right. We need a mutex here because for a brief while we will - have the situation that fd is actually closed. Hence if a - second thread were to get into this block, its dup() would - likely return our fd as its dupfd. (after all, it is closed). + /* Right. We need a mutex here because for a brief while we + will have the situation that fd is actually closed. Hence if + a second thread were to get into this block, its dup() would + likely return our fd as its dupfd. (after all, it is closed) Then if we get to the dup2() first, we blat the fd back (messing up its temporary as a side effect) only for it to then close its dupfd (== our fd) in its close(dupfd) */ /* There is, of course, a race condition, that any other thread trying to input/output/whatever on this fd will be stuffed - for the duraction of this little manoeuver. Perhaps we should - hold an IO mutex for the duration of every IO operation if - we know that invalidate doesn't work on this platform, but - that would suck, and could kill performance. + for the duration of this little manoeuvrer. Perhaps we + should hold an IO mutex for the duration of every IO + operation if we know that invalidate doesn't work on this + platform, but that would suck, and could kill performance. Except that correctness trumps speed. Advice from klortho #11912. */ |