[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
| Re: [ecf-dev] Resume and file append | 
Hi Thomas,
Thomas Hallgren wrote:
Hi,
I'm just looking at the IIncomingFileTransferReceiveResumedEvent and 
its receive methods. It strikes me as a bit odd that the two methods 
that takes a File as a parameter will truncate this file rather then 
appending to it. Why is that? 
I think perhaps for consistency (i.e. with the 
IIncomingFileTransferReceiveStartEvent methods).  To be honest, though, 
I can't remember for sure.
Wouldn't it be more intuitive to open the file for append when a 
paused transfer is resumed? Perhaps the actual append flag that can be 
passed to the FileOutputStream constructor should be included as a 
parameter to receive?
This seems reasonable to me.  I guess if we are going to add this there 
is a choice:
1) Change the method IIFTRRE.receive(File) to 
IIFTRRE.receive(File,boolean append)
OR 2) Add method IIFTRRE.receive(File,boolean append) and leave 
receive(File) (and File, FileTransferJob) the same.
I'm sort of inclined to choose '1' even though this is a breaking API 
change as going forward this will be simpler.  Any thoughts/comments?
Scott
Kind Regards,
Thomas Hallgren
_______________________________________________
ecf-dev mailing list
ecf-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/ecf-dev