On Thu, Jan 21, 2016 at 12:35 AM, Rainer Jung <rainer.j...@kippdata.de> wrote:
> Am 21.01.2016 um 00:17 schrieb Yann Ylavic:
>>
>> On Tue, Jan 19, 2016 at 4:45 PM,  <rj...@apache.org> wrote:
>>>
>>> Author: rjung
>>> Date: Tue Jan 19 15:45:44 2016
>>> New Revision: 1725551
>>>
>>> URL: http://svn.apache.org/viewvc?rev=1725551&view=rev
>>> Log:
>>> Improve spatch for APLOGNO a bit.
>>>
>>> Modified:
>>>      httpd/httpd/trunk/docs/log-message-tags/find-messages.cocci
>>>
>>> Modified: httpd/httpd/trunk/docs/log-message-tags/find-messages.cocci
>>> URL:
>>> http://svn.apache.org/viewvc/httpd/httpd/trunk/docs/log-message-tags/find-messages.cocci?rev=1725551&r1=1725550&r2=1725551&view=diff
>>>
>>> ==============================================================================
>>> --- httpd/httpd/trunk/docs/log-message-tags/find-messages.cocci
>>> (original)
>>> +++ httpd/httpd/trunk/docs/log-message-tags/find-messages.cocci Tue Jan
>>> 19 15:45:44 2016
>>> @@ -39,9 +39,10 @@ identifier level =~ "^APLOG_(EMERG|ALERT
>>>   expression rv, p, s, cert;
>>>   constant char[] fmt !~ "^APLOGNO";
>>>   identifier level =~
>>> "^APLOG_(EMERG|ALERT|CRIT|ERR|WARNING|NOTICE|INFO|STARTUP|DEBUG)$";
>>> +identifier fn =~ "^ssl_log_x(|r|c)error$";
>>
>>
>> Typo "ssl_log_x(|r|c)" => "ssl_log_(x|r|c)"?
>
>
> Close: the functions are
>
> - ssl_log_xerror
> - ssl_log_cxerror
> - ssl_log_rxerror

Ah, right.

>
> so "xerror" is the fixed suffix. Should be fixed now by r1725837.

"^ssl_log_(|r|c)xerror$" => "^ssl_log_(r|c)?xerror$"?
Or is it some special coccinelle regex syntax?

Thanks,
Yann.

Reply via email to