On Thu, 2009-12-17 at 00:43 +0300, Aleksej Saushev wrote: > Hello! > > I need this for both, FriCAS and OpenAxiom, to build last released versions, > I also think it is good idea anyway: > > $NetBSD: patch-ab,v 1.1 2009/12/15 23:36:29 asau Exp $ > > Reuse strcmp to simplify logic and fix build. > Shared between FriCAS and OpenAxiom. > > --- src/hyper/addfile.c.orig 2009-07-10 00:50:28.000000000 +0400 > +++ src/hyper/addfile.c 2009-12-15 22:32:55.000000000 +0300 > @@ -50,13 +50,7 @@ > strpostfix(char *s, char *t) > { > int slen = strlen(s), tlen = strlen(t); > - > - if (tlen > slen) > - return 0; > - while (tlen > 0) > - if (s[--slen] != t[--tlen]) > - return 0; > - return 1; > + return (tlen < slen) && !strcmp(s+slen-tlen, t);
I have no idea how this is used, or what is correct, but should the case tlen == slen always return false? > } > > /* extend_ht : just checks the name and adds a .ht if needed */ > > ------------------------------------------------------------------------------ This SF.Net email is sponsored by the Verizon Developer Community Take advantage of Verizon's best-in-class app development support A streamlined, 14 day to market process makes app distribution fast and easy Join now and get one step closer to millions of Verizon customers http://p.sf.net/sfu/verizon-dev2dev _______________________________________________ open-axiom-devel mailing list open-axiom-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/open-axiom-devel