httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gregg Smith <...@gknw.net>
Subject Re: proposed backport of mod_h2 - v4
Date Sat, 05 Sep 2015 07:36:20 GMT
In core-h2-all-in-one-v4.patch line 8586
+        if (!offers || ap_array_contains(offers, *protos)) {
the old ap_array_contains is in use

On 9/5/2015 12:21 AM, Stefan Eissing wrote:
> yes, it should. where do you not see it?
>
> (i wish we'd use branches instead of this patch file madness)
>
>> Am 05.09.2015 um 06:12 schrieb Gregg Smith<gls@gknw.net>:
>>
>> Shouldn't this be ap_array_str_contains now in h2_switch.c?
>>
>> +    while (*protos) {
>> +        /* Add all protocols we know (tls or clear) and that
>> +         * are part of the offerings (if there have been any).
>> +         */
>> -->+        if (!offers || ap_array_contains(offers, *protos)) {
>> +            ap_log_cerror(APLOG_MARK, APLOG_TRACE1, 0, c,
>> +                          "proposing protocol '%s'", *protos);
>>
>>
>>
>>


Mime
View raw message