Re: [patch] fork optional branch point normazilation

From: Brad Roberts <braddr@puremagic.com>
Date: Apr 18, 2005, 02:01 UTC
Message-ID: <Pine.LNX.4.44.0504171859410.2625-100000@bellevue.puremagic.com>
In-Reply-To: <Pine.LNX.4.58.0504171806230.7211@ppc970.osdl.org>

The patch needed some tweaking, but not in the way you thought. :)

commit a6aa192641e9ea242332fee4916abf5ad2640d75 tree c69878b009ec2f505d75aa7d99e9ee30cd21ab02 parent 60e1274460f50bcecdc3f162b4fced9e5ebf2dfb author Brad Roberts <braddr@puremagic.com> 1113789519 -0700 committer Brad Roberts <braddr@gameboy2> 1113789519 -0700

Fix remove_specials for real. The second half logic needs the original head of the string.

Signed-off-by: Brad Roberts <braddr@puremagic.com>

Index: commit-tree.c
===================================================================
--- 02cf6917da6297ff4f9172f7af174ba329f01b3d/commit-tree.c  (mode:100644 sha1:ec53a4565ec0033aaf6df2a48d233ccf4823e8b0)
+++ c69878b009ec2f505d75aa7d99e9ee30cd21ab02/commit-tree.c  (mode:100644 sha1:f41cda6f9496b9e33cb95305ef1093f846c663ae)
@@ -83,11 +83,11 @@
 static void remove_special(char *p)
 {
        char c;
-       char *dst = p;
+       char *dst = p, *src = p;

        for (;;) {
-               c = *p;
-               p++;
+               c = *src;
+               src++;
                switch(c) {
                case '\n': case '<': case '>':
                        continue;


Later,
Brad