On 24/08/2015 7:44 a.m., Amos Jeffries wrote:
> Okay. Added the flushing but left the existing StoreEntry logic bug in
> place.
>
> Also re-added the flush/buffer unit testing since that is now Packable
> API relevant.
>
> Everything that auditor script you posted complains about now is
> intenti
Okay. Added the flushing but left the existing StoreEntry logic bug in
place.
Also re-added the flush/buffer unit testing since that is now Packable
API relevant.
Everything that auditor script you posted complains about now is
intentional whitespace polish.
In the side topic of Big3 rules this
On 08/22/2015 04:18 AM, Amos Jeffries wrote:
> On 22/08/2015 5:44 a.m., Alex Rousskov wrote:
>> On 08/21/2015 04:38 AM, Amos Jeffries wrote:
>>> So PackableStream is really just a rename of StoreEntryStream BUT with
>>> some implicit new properties from the underlying type change:
>>>
>>> * lack of
On 22/08/2015 5:44 a.m., Alex Rousskov wrote:
> On 08/21/2015 04:38 AM, Amos Jeffries wrote:
>> So PackableStream is really just a rename of StoreEntryStream BUT with
>> some implicit new properties from the underlying type change:
>>
>> * lack of Store.h dependency
>>
>> * ability to stream into a
On 08/21/2015 04:38 AM, Amos Jeffries wrote:
> So PackableStream is really just a rename of StoreEntryStream BUT with
> some implicit new properties from the underlying type change:
>
> * lack of Store.h dependency
>
> * ability to stream into a MemBuf if its creator desires that
>
> This patch
So PackableStream is really just a rename of StoreEntryStream BUT with
some implicit new properties from the underlying type change:
* lack of Store.h dependency
* ability to stream into a MemBuf if its creator desires that
This patch demonstrates that by doing the upgrade in-place and re-typing