Re: [twsocket] [PATCH] THttpSrv: add support for returning partialstreams

2012-02-05 Thread Arno Garrels
RTT wrote: On 05-02-2012 12:15, Arno Garrels wrote: So there must be an option to allow ranges on a per request basis. A new method would provide such an option. I see. A new method then (and AnswerStreamAcceptRange seems more appropriate). Or, probably better, replace the AnswerStream

Re: [twsocket] [PATCH] THttpSrv: add support for returning partialstreams

2012-02-05 Thread RTT
On 05-02-2012 17:38, Arno Garrels wrote: Problem is that AnswerStreamAcceptRange can only be used for OK-responses. If I'm understanding this correctly, can't the status parameter be added, and then response handled accordingly? Right now I don't need this functionality. I'm just trying to

Re: [twsocket] [PATCH] THttpSrv: add support for returning partialstreams

2012-02-05 Thread Tobias Rapp
Arno Garrels wrote (in a response to RTT): Problem is that AnswerStreamAcceptRange can only be used for OK-responses. Yes, that is right and like you said I only use it for some of the responses where it makes sense for the client (to allow seeking within streamed audio files). I just

Re: [twsocket] [PATCH] THttpSrv: add support for returning partialstreams

2012-02-04 Thread Arno Garrels
Tobias Rapp wrote: Hi! I have added a new function AnswerStreamPart Shouldn't that method be named AnswerStreamAcceptRange, or at least AnswerStreamRange? -- Arno Garrels in component THttpConnection to allow answering partial retrieve requests using any tStream descendant (see

Re: [twsocket] [PATCH] THttpSrv: add support for returning partialstreams

2012-02-04 Thread RTT
On 04-02-2012 11:25, Arno Garrels wrote: Tobias Rapp wrote: Hi! I have added a new function AnswerStreamPart Shouldn't that method be named AnswerStreamAcceptRange, or at least AnswerStreamRange? How about if the this ranges handling code is added to the AnswerStream method? Why and