On Wed, May 02, 2001 at 08:03:24PM -0500, Brandon wrote: > > > But you are the author of SimplifiedClient after all.. can you comment > > on whether SimplifiedClient would be affected by my proposed changes to > > the client events? Of course I realize it would need to be tweaked if > > we dispensed with the TRANSFERRING Request state. > > I suppose you mean will it break it fundamentally since obviously it will > have to be tweaked whenever you change the Client API around. The only > thing that might be broken by changing the events around is that you have > to make sure to allow for streaming. So there needs to be an event that is > throw when the metadata has finished downloading but before the data has > started downloading, and of course an event needs to be thrown when the > data has finished downloading. For uploading this isn't necessary since > you have to have the entire file before the transfer can start. > > But basically it would be pretty hard to change Client so that > SimplifiedClient wouldn't work since it abstracts the events to a very > high level. The only reason I mention the metadata finished event is > because this wasn't originally included in the Client API and had to be > added to facilitate streaming. If there was such an oversight in the > beginning then it's the sort of thing that might be phased out during > simplification, so I thought I'd better mention it, just in case.
Well, as I discussed, I only wish to remove the events RequestCompleteEvent, RequestSuccessfulEvent, and RequestFailedEvent, which are either unused or redundant, and to eliminate the TRANSFERRING Request state. There is no need to mess with the Transfer* events or the SegmentCompleteEvent (which is the event triggered when the metadata is finished downloading). -- # tavin cole # # "The process of scientific discovery is, in effect, # a continual flight from wonder." # - Albert Einstein _______________________________________________ Devl mailing list Devl at freenetproject.org http://lists.freenetproject.org/mailman/listinfo/devl
