Okay by me. We're not using these in the new cache. You might want to wait until
we've completely transitioned to the new cache, unless you don't mind fixing up the
old cache to keep working.
Comments from the rest of the cache crowd?
Gordon
Brendan Eich wrote:
> Its implementation is just wrong: it reads one character at a time,
> because nsIBinaryOutputStream::WriteStringZ's implementation writes a
> nul-terminated string. I plan to change both to stream a counted
> string. I see the old cache using these methods
> (http://lxr.mozilla.org/mozilla/source/netwerk/cache/mgr/nsCachedNetData.cpp#493,
> http://lxr.mozilla.org/mozilla/source/netwerk/cache/mgr/nsCachedNetData.cpp#812),
> but if those are the only uses, I argue we should break API
> compatibility here and get rid of this single-byte-read nonsense. Comments?
>
> /be
