Am Mittwoch, 19. November 2014, 22:21:12 schrieb Darshit Shah: > On 11/18, Tim Rühsen wrote: > >I amended three tests to fail when run with turkish locale. > >I fixed these issues (using c_strcasecmp/c_strncasecmp) and also replaced > >strcasecmp/strncasecmp by c_strcasecmp/c_strncasecmp at places where we > >definitely want a ASCII comparison instead of a locale dependent one. > > > >There are still some places left where we use strcasecmp/strncasecmp, e.g. > >domain/host and filename comparisons. > > > >Please have a look... > > > >Tim > > In cookies.c, the header file is added on the same line as the ^L Page break > character. Please add the file before the Page Break. > > I'm assuming your editor didn't show you that character, but git diff does. > Try to enable such invisible characters in your editor.
No, you are right. Vi shows it, my IDE does not. Dumb question: I never saw ^L in source code. What is it good for ? Printing ? > Apart from that, it looks like a good patch. Please push it when you can Thanks, I'll have some time in a few hours. Tim
signature.asc
Description: This is a digitally signed message part.
