[Mono-dev] [Mono-patches] r108492 - in branches/mono-2-0/mcs/class/System.Web: System.Web.UI Test/System.Web.UI

[Dean Brettle]

On Tue, Jul 22, 2008 at 12:12 PM, Gert Driesen <gert.driesen at telenet.be>
wrote:

> Also, your patch was not complete; Stream.Length can/will also throw a
> NotSupportedException when the stream is unseekable. I'll commit a slighty
> modified version to SVN HEAD in a few minutes.


I'm not sure what you man by unseekable vs CanSeek == false.  You mean it is
possible for Stream.Length to throw an exception even if CanSeek== true?  If
so, I guess you are going to handle that case with a try/catch.  FWIW, if we
are going to add a try/catch for the unseekable case, I recommend making it
also handle MemoryStreams which are not publicly accessible.  The code
currently throws an exception in that case as well.

--Dean
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.ximian.com/pipermail/mono-devel-list/attachments/20080722/505857c4/attachment.html