Mailing List Archive: Cherokee: dev

cherokee_strlcat

Index | Next | Previous | View Threaded

aperez at skarcha

Sep 13, 2009, 1:35 AM

Post #1 of 3 (354 views)
Permalink

cherokee_strlcat

Hello!

I've found what I think is duplicated code in "cherokee/utils.c"
related to "strlcat". I guess that "cheroke_strlcat" could be removed,
actually.

Please, see attached patch.

--
Saludos:
Antonio Pérez

Attachments:

strlcat.diff (3.64 KB)

alvaro at octality

Sep 13, 2009, 2:29 AM

Post #2 of 3 (335 views)
Permalink

Re: cherokee_strlcat [In reply to]

On 13-sep-09, at 10:35, Antonio Pérez wrote:

> I've found what I think is duplicated code in "cherokee/utils.c"
> related to "strlcat". I guess that "cheroke_strlcat" could be removed,
> actually.

Good catch!

> Please, see attached patch.

+1 :)

--
Octality
http://www.octality.com/

_______________________________________________
Cherokee-dev mailing list
Cherokee-dev [at] lists
http://lists.octality.com/listinfo/cherokee-dev

aperez at skarcha

Sep 13, 2009, 4:52 PM

Post #3 of 3 (318 views)
Permalink

Re: cherokee_strlcat [In reply to]

Hello,

2009/9/13 Alvaro Lopez Ortega <alvaro [at] octality>:

> On 13-sep-09, at 10:35, Antonio Pérez wrote:
>
>> I've found what I think is duplicated code in "cherokee/utils.c"
>> related to "strlcat". I guess that "cheroke_strlcat" could be removed,
>> actually.
>
> Good catch!
>
>> Please, see attached patch.
>
> +1 :)

Applied in r3659 :-)

--
Saludos:
Antonio Pérez
_______________________________________________
Cherokee-dev mailing list
Cherokee-dev [at] lists
http://lists.octality.com/listinfo/cherokee-dev

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads