On 08/30/2013 02:10 PM, Anoop Saldanha wrote:
> In src/detect-engine-dcepayload.c:
> 
>> @@ -9483,6 +9483,9 @@ int DcePayloadParseTest41(void)
>>  
>>  /**
>>   * \test Test the working of consecutive relative matches with a negated content.
>> + *
>> + * By AWS.  This test was modified by a commit
>> + * a8d401a5011374570b6f090a21dc65d704c64e38.
> 
> If I am going through the unittests separately tomorrow, I have no
> reference about what went on through the unittests. The message gives me
> a reference and I think the commit message gave it sufficient info.

It doesn't make sense to add a commit reference/changelog then. Add a
reference to the bug ticket or explain the test case directly. If you
want to know what commit changed the code, just use git blame. Thats why
we use version control.