]> Lady’s Gitweb - Gitweb/commit
gitweb: remove unnecessary test when closing file descriptor
authorSylvain Rabot <redacted>
Thu, 30 Dec 2010 21:20:29 +0000 (22:20 +0100)
committerLady <redacted>
Mon, 6 Apr 2026 04:51:30 +0000 (00:51 -0400)
commit6a1ba015e29ba7ad66fcc57309941cb92a024a32591ac3860d4ce7a47daaeca1
treeb540bc566e3807be4f5d6aefb4026136e67d1886204355ba590ff11a293f7a51
parent89c1fb3478173d8eb5633f41cb98da4aa31f30fbed58573f5ca01972ded20af8
gitweb: remove unnecessary test when closing file descriptor

It happens that closing file descriptor fails whereas the blob is
perfectly readable.  According to perlman the reasons could be:

   If the file handle came from a piped open, "close" will additionally
   return false if one of the other system calls involved fails, or if the
   program exits with non-zero status.  (If the only problem was that the
   program exited non-zero, $! will be set to 0.)  Closing a pipe also waits
   for the process executing on the pipe to complete, in case you want to
   look at the output of the pipe afterwards, and implicitly puts the exit
   status value of that command into $?.

   Prematurely closing the read end of a pipe (i.e. before the process writ-
   ing to it at the other end has closed it) will result in a SIGPIPE being
   delivered to the writer.  If the other end can't handle that, be sure to
   read all the data before closing the pipe.

In this case we don't mind that close fails.

Signed-off-by: Sylvain Rabot <redacted>
Signed-off-by: Junio C Hamano <redacted>
gitweb.perl
This page took 0.234968 seconds and 5 git commands to generate.