Re: memory corruption bug fix, libpcp p_instance.c

[fche@xxxxxxxxxx (Frank Ch. Eigler)]

Ken McDonell <kenj@xxxxxxxxxxxxxxxx> writes:

> [...]
>> valgrind pmval 'pmcd.buf.alloc' -i \
>>    `awk 'BEGIN { for (i=0;i<3000;i++) { printf "x" }; printf("\n"); }'`
>
> This is a classical example of where a unit test should _not_ be
> added to an existing test, rather a new test should be created.  If
> we added it to an existing test that would fail until the code fix
> is applied [...]

OK (though in this case the code fix came at the same time).


> [...]
> Making a new test is so simple ...

Ah, neat, and it's mentioned in qa/README already.


- FChE