On 12/12/05, Greg Hudson <ghudson@mit.edu> wrote:
> On consideration, here is where I would like us to be:
>
> * Streams which read from or write to an underlying object own that
> object, i.e. closing the stream closes the underlying object, if
> applicable.
>
> * The layer (function or data type) which created a stream is
> responsible for closing it, except when the above rule applies.
>
> * Window handlers are thought of as an odd kind of stream, and passing
> the final NULL window is considered closing the stream.
>
> If you think of apply_textdelta as creating a window handler, then I
> don't think we're too far off. svn_stream_from_aprfile isn't owning its
> subsidiary file, svn_txdelta_apply is erroneously taking responsibility
> for closing the window stream it is passed, and there may be some other
> deviations.
Well, I could create an svn_stream_from_aprfile2 which conforms to the
above model. Whether svn_stream_from_aprfile should be immediately
deprecated is not clear to me, but we might as well.
Would it be a good idea to store the above in a text file (in notes/
for example), for later retrieval?
bye,
Erik.
Received on Mon Dec 12 23:05:13 2005