U++ forum: Welcome to the forum

Re: PipeStream - bidirectional Stream [message #38945 is a reply to message #38944]

Mon, 28 January 2013 21:08

dolik.rce
Messages: 1789
Registered: August 2008
Location: Czech Republic

Ultimate Contributor

Hi Mirek

mirek wrote on Mon, 28 January 2013 20:04

Looking at the code, I am not quite sure that you interpret loading/storing flag correctly. It really is just a flag for serialization routines...

I'm well aware that I give the loading/storing flag completely new meaning. A separate flag could be introduced to get the bidirectional access, but I don't think there would be much difference...

mirek wrote on Mon, 28 January 2013 20:04

I think I would not bother implementing SetLoading / SetStoring in PipeStream. I believe that in typical scenario, only either input or output will do serialization, so it makes sense to set in just for single end of pipe... In that case, what is already in Stream should be enough.

Perhaps you are right about this. I can't really imagine any scenario where PipeStream would be used for serialization. It is intended for different tasks, mainly as buffer between various interfaces, as mentioned above.

Also, since the last version doesn't require explicitly calling SetLoading/SetStoring before reading/writing (first versions did treat incorrect state as error), and since using it for serialization is unlikely, it can be probably dropped without loosing any capabilities. Tomorrow I will update the code and than we can discuss it further...

Thanks for your feedback.

Honza

Report message to a moderator