Thanks Ryan, this will definitely not leak handles. I had thought about making a strict version of hGetContents, though on a bit different lines.<br><br>My question was that since the documentation says that the semi-closed handle becomes closed as soon as the entire contents have been read; can I conclude that as far as I consume the string, I am not leaking handles?<br>
<br>I am still interested in using hGetContents, since these contents are going soon through hPutStr, which will consume it anyway. And hGetContents being lazy will not occupy memory of the order of size of the input file. That's why the question.<br>
<br>Regards,<br>Abhay<br><br><div class="gmail_quote">On Tue, Apr 15, 2008 at 1:07 PM, Ryan Ingram <<a href="mailto:ryani.spam@gmail.com">ryani.spam@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I usually use something like this instead:<br>
<br>
hStrictGetContents :: Handle -> IO String<br>
hStrictGetContents h = do<br>
s <- hGetContents h<br>
length s `seq` hClose h<br>
return s<br>
<br>
This guarantees the following:<br>
1) The whole file is read before hStrictGetContents exits (could be<br>
considered bad, but usually it's The Right Thing)<br>
2) You guarantee that you don't leak file handles (good benefit!)<br>
<br>
A slightly better version:<br>
<br>
import qualified Data.ByteString.Char8 as B<br>
<br>
hStrictGetContents :: Handle -> IO String<br>
hStrictGetContents h = do<br>
bs <- B.hGetContents h<br>
hClose h -- not sure if this is required; ByteString documentation<br>
isn't clear.<br>
return $ B.unpack bs -- lazy unpack into String<br>
<br>
This saves a ton of memory for big reads; a String is ~12 bytes per<br>
character, this is only 1 byte per character + fixed overhead. Then,<br>
assuming the function consuming the String doesn't leak, you'll end up<br>
with a much smaller space requirement.<br>
<br>
-- ryan<br>
<div><div></div><div class="Wj3C7c"><br>
2008/4/14 Abhay Parvate <<a href="mailto:abhay.parvate@gmail.com">abhay.parvate@gmail.com</a>>:<br>
> Thanks! I was worried about how/where would I place hClose!<br>
><br>
><br>
><br>
> On Mon, Apr 14, 2008 at 10:58 PM, Brent Yorgey <<a href="mailto:byorgey@gmail.com">byorgey@gmail.com</a>> wrote:<br>
> ><br>
> ><br>
> > 2008/4/14 Abhay Parvate <<a href="mailto:abhay.parvate@gmail.com">abhay.parvate@gmail.com</a>>:<br>
> ><br>
> ><br>
> ><br>
> ><br>
> > > Hello,<br>
> > ><br>
> > > In describing the Handle type, the GHC documentation says (in the<br>
> System.IO documentation):<br>
> > ><br>
> > ><br>
> > > GHC note: a Handle will be automatically closed when the garbage<br>
> collector detects that it has become unreferenced by the program. However,<br>
> relying on this behaviour is not generally recommended: the garbage<br>
> collector is unpredictable. If possible, use explicit an explicit hClose to<br>
> close Handles when they are no longer required. GHC does not currently<br>
> attempt to free up file descriptors when they have run out, it is your<br>
> responsibility to ensure that this doesn't happen.<br>
> > ><br>
> > > But one cannot call hClose on Handles on which something like<br>
> hGetContents has been called; it just terminates the character list at the<br>
> point till which it has already read. Further the manual says that<br>
> hGetContents puts the handle in the semi-closed state, and further,<br>
> > ><br>
> > ><br>
> > > A semi-closed handle becomes closed:<br>
> > ><br>
> > > if hClose is applied to it;<br>
> > > if an I/O error occurs when reading an item from the handle;<br>
> > > or once the entire contents of the handle has been read. So do I safely<br>
> assume here, according to the third point above, that it's fine if I do not<br>
> call hClose explicitly as far as I am consuming all the contents returned by<br>
> hGetContents?<br>
> > ><br>
> ><br>
> ><br>
> > Yes, not only is it fine, it's recommended! Calling hClose explicitly on<br>
> a handle after calling hGetContents is a sure way to introduce bugs.<br>
> ><br>
> > -Brent<br>
> ><br>
> ><br>
><br>
><br>
</div></div>> _______________________________________________<br>
> Haskell-Cafe mailing list<br>
> <a href="mailto:Haskell-Cafe@haskell.org">Haskell-Cafe@haskell.org</a><br>
> <a href="http://www.haskell.org/mailman/listinfo/haskell-cafe" target="_blank">http://www.haskell.org/mailman/listinfo/haskell-cafe</a><br>
><br>
><br>
</blockquote></div><br>