[BFAV] "Couldn't delete old version of file" exception revisited

Frank Shearar frank.shearar at rnid.org.uk
Thu Feb 26 17:05:37 UTC 2004


> >>> "Brent Vukmer" 02/26/04 16:36 >>>

Me:
> > Does BFAV keep a FileStream open for its entire life, or just 
> > for the processes involving reading/writing? I mean, when you 
> > hit load updates, does BFAV then open a FileStream on 
> > latest.zip and then clean up that FileStream after?
> > 
> 
> It *should* be doing that; perhaps there is not cleanup 
> ensure: block in every place there ought to be.  

Do you mean that, for instance, ArchiveRepository>>downloadZips:
should have

	file _ self repositoryDir forceNewFileNamed: fn.
	[file binary nextPutAll: response bodyBytes.]
		ensure: [file close].
	downloads add: fn.

instead of

	file _ self repositoryDir forceNewFileNamed: fn.
	file binary nextPutAll: response bodyBytes.
	file close.	
	downloads add: fn.

?

frank
-------------- next part --------------
A non-text attachment was scrubbed...
Name: winmail.dat
Type: application/ms-tnef
Size: 1912 bytes
Desc: not available
Url : http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20040226/74025c06/winmail.bin


More information about the Squeak-dev mailing list