Use a binary 0 as string separator

This makes things not break horribly when we marshall strings containing ";". However, I left the TODO, as we'll eventually want something more robust.

[Wladimir Palant, 2013-05-27 09:43:13]

I don't really like the approach (and zero-terminated strings in general).
Strings need to be serialized by writing out their length as (16 bit?) integer,
followed by the string data.

[Oleksandr, 2013-05-27 14:54:50]

On 2013/05/27 09:43:13, Wladimir Palant wrote:
> I don't really like the approach (and zero-terminated strings in general).
> Strings need to be serialized by writing out their length as (16 bit?)
integer,
> followed by the string data.

I think if we go that way it would be better just to use BSTR and
SysAllocString/SysFreeString.

[Wladimir Palant, 2013-05-27 15:22:43]

On 2013/05/27 14:54:50, Oleksandr wrote:
> I think if we go that way it would be better just to use BSTR and
> SysAllocString/SysFreeString.

I don't really see the advantage here. I would rather create a proper
serialization buffer that can hold different data types - going to look into
that tomorrow.

[Oleksandr, 2013-05-27 16:28:17]

On 2013/05/27 15:22:43, Wladimir Palant wrote:
> On 2013/05/27 14:54:50, Oleksandr wrote:
> > I think if we go that way it would be better just to use BSTR and
> > SysAllocString/SysFreeString.
> 
> I don't really see the advantage here. I would rather create a proper
> serialization buffer that can hold different data types - going to look into
> that tomorrow.

In that case I'd suggest using IStream objects (using CreateStreamOnHGlobal) and
this article in general http://msdn.microsoft.com/en-us/magazine/cc301563.aspx.
I was going to look at it as well as soon as traverser is ready.

[Felix Dahlke, 2013-06-03 10:19:10]

On 2013/05/27 09:43:13, Wladimir Palant wrote:
> I don't really like the approach (and zero-terminated strings in general).
> Strings need to be serialized by writing out their length as (16 bit?)
integer,
> followed by the string data.

Agreed, it was just a hack. Closing this since we have a proper implementation
now: http://codereview.adblockplus.org/10824027/