Menu

#25 Constructor CompoundFile(Stream stream, leaveOpen=true)

v2.0
closed-fixed
nobody
None
5
2016-12-29
2014-12-23
No

Hi!

Please, add leaveOpen parameter to constructor CompoundFile(Stream stream). My code opens stream and my code responsible for close it. As an analogy for BinaryReader constructor (http://msdn.microsoft.com/ru-ru/library/gg712804(v=vs.110).aspx).

Check my code on GitHub: https://github.com/KLSRL/OpenMCDF/commit/55f73b162a0f9f44a7b5d44ea9d85613eaaa0255

Discussion

  • Federico

    Federico - 2014-12-24

    I hope that adding parameter 'closeStream' on Close() method works for you.
    If that's not true, please let me know.
    Commited to dev.

    Best Regards,
    Federico

     
  • Federico

    Federico - 2014-12-24
    • status: open --> accepted
     
  • Pavel Martynov

    Pavel Martynov - 2014-12-24

    I'm afraid it does not solve my problem. My case: constructor throws exception (for example, if file corrupted) and closes stream, but I want to use this stream latetly:

    var badStream = OpenStream();
    try
    {
        var cf = new CompoundFile(badStream); // throws exception
    }
    catch
    {
        // badStream there closed by CompoundFile, but I want use it and cant reopen it
    }
    
     

    Last edit: Pavel Martynov 2014-12-25
  • Federico

    Federico - 2016-12-29
    • status: accepted --> closed-fixed
     
  • Federico

    Federico - 2016-12-29

    Closed after github migration
    Best Regards,
    Federico

     

Log in to post a comment.