Looks good to me.
Thanks,
Hans
On Tue, 2011-05-03 at 01:58 +0900, Stephen J. Turnbull wrote:
APPROVE COMMIT 21.5
Recommended by Hans de Graaff of Gentoo and reviewed by Marcus
Crestani. Hans, Marcus, please check that I've done the right thing
here. I'll close issue 354 after confirmation, or Marcus can do it
since it's assigned to him.
Marcus, thanks for the review, and thanks to Aidan and Hans for the
analysis.
diff -r 70af4098e14f src/ChangeLog
--- a/src/ChangeLog Mon May 02 10:51:26 2011 +0100
+++ b/src/ChangeLog Tue May 03 01:42:33 2011 +0900
@@ -1,3 +1,8 @@
+2011-05-03 Stephen J. Turnbull <stephen(a)xemacs.org>
+
+ * dumper.c (pdump_file_try): Remove static qualifier.
+ Thanks to Hans de Graaff of Gentoo and Marcus Crestani.
+
2011-05-01 Aidan Kehoe <kehoea(a)parhasard.net>
* lread.c (parse_integer):
diff -r 70af4098e14f src/dumper.c
--- a/src/dumper.c Mon May 02 10:51:26 2011 +0100
+++ b/src/dumper.c Tue May 03 01:42:33 2011 +0900
@@ -2593,8 +2593,9 @@
#endif /* !WIN32_NATIVE */
-
-static int
+/* This used to be static, but there seems to be a bug in the GCC 4.1.2
+ optimizer that clobbers exe_path. */
+int
pdump_file_try (Wexttext *exe_path)
{
Wexttext *w = exe_path + wext_strlen (exe_path);
_______________________________________________
XEmacs-Patches mailing list
XEmacs-Patches(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-patches