Ticket #25 (defect)

Opened 3 years ago

Last modified 1 year ago

Use File::Copy to copy physical file contents

Status: closed (fixed)

Reported by: toby Assigned to: toby
Priority: major Milestone: 0.11.0
Component: vss2svn.pl Version: 0.11.0
Keywords: Cc:

source:/trunk/script/Vss2Svn/Dumpfile.pm get_export_contents() currently reads the entire contents of exported VSS files into memory, which could potentially cause memory usage problems for repos containing many large files.

File::Copy should be used instead, since it uses 2M buffered reads.

Attachments

Dumpfile.pm.diff (0.9 kB) - added by toby on 12/07/2006 09:51:25 AM.
Patch from George Banfill to do a buffered output of file contents; full file is still read into memory

Change History

12/07/2006 09:51:25 AM: Modified by toby

  • attachment Dumpfile.pm.diff added.

Patch from George Banfill to do a buffered output of file contents; full file is still read into memory

02/11/2007 02:23:07 AM: Modified by toby

  • status changed from new to assigned.
  • version changed from 0.10.0 to 0.11.0.
  • milestone changed from 0.10 to 0.11.0.

06/11/2007 10:41:21 PM: Modified by toby

  • status changed from assigned to closed.
  • resolution set to fixed.

Fixed in r312 using patch from Erik Bray.

PumaCode.org recommends CVSDude for fast, professional Subversion and Trac hosting:

CVSDude.com

These ads are automatically generated by Google. Revenue from these ads helps to pay for hosting this site; however, these ads do not constitute an endorsement by PumaCode.org.