strcasecmp_m() in source3

Andrew Bartlett abartlet at samba.org
Wed May 18 16:12:56 MDT 2011


On Wed, 2011-05-18 at 10:25 -0700, Jeremy Allison wrote:
> On Tue, May 17, 2011 at 12:53:04AM +1000, Andrew Bartlett wrote:
> > Jeremy,
> > 
> > We discussed my proposal to move the source3 code to always using the
> > codepoints base strcasecmp_m() code (which never allocates, and has an
> > ASCII fast path).  Attached are two patches that do that.
> > 
> > What do you think?
> > 
> > If you want more tests, I am looking to revive the currently unsued
> > comfychair based python tests for this code, but the strcasecmp_m
> > function is already the basis for the source4 code, has tests, and is
> > well tested in that use. 
> > Andrew Bartlett
> 
> I was planning to review this, but I guess you couldn't wait :-(.
> 
> Next time give me a heads up before you just push something please.
> 
> I know I have a lot of other stuff going on but would have liked to know
> you were planning to push. If I don't have time I won't block you from
> pushing anyway, but it's polite to keep a requested reviewer in the
> loop.
> 
> If I didn't care about this change, I wouldn't have asked to be a
> reviewer.

I'm sorry.  It was very rude on my part and I do apologise.

Andrew Bartlett

-- 
Andrew Bartlett                                http://samba.org/~abartlet/
Authentication Developer, Samba Team           http://samba.org



More information about the samba-technical mailing list