Re: [BUG] http.c (http_something_accepted())
Stipe Tolj <[email protected]>
| Newsgroups | gmane.comp.mobile.kannel.devel |
|---|---|
| Organization | Wapme Systems AG |
| Message-ID | <[email protected]> |
Vjacheslav Chekushin wrote: > > Hi, list. > > octstr_compare() can't be used because http headers are packed by http_pack() > and headers can contain many values. > So octstr_search() must be used. ok, you consider that you have multi-valued headers, like for the 'Accept-Charset' headers, right? That would be a problem, yes. But actually I just wanted to get rid of the istrup() function and use Ocstr* instead. I have not claimed that it would now work even for multi-value headers. Can you preliminary file this as a bug report to http://bugs.kannel.org/ so we keep track of it?! Thanks. > By the way, why do you need octstr_convert_range() in this function if we could > use octstr_case_search() ? That's a good question?! ;) seems you're right here, of course. It's unnecessary to upcase both operands in order to have a case sensitive search while there is an incase sensitive search function available. I'll change this. Stipe [email protected] ------------------------------------------------------------------- Wapme Systems AG Vogelsanger Weg 80 40470 Düsseldorf Tel: +49-211-74845-0 Fax: +49-211-74845-299 E-Mail: [email protected] Internet: http://www.wapme-systems.de ------------------------------------------------------------------- wapme.net - wherever you are