commit/XEmacs: kehoea: Increase READ_BUF_SIZE, used in #'insert-file-contents-internal, to #x10000

[email protected]
Newsgroups gmane.emacs.xemacs.patches
Message-ID <[email protected]>
1 new commit in XEmacs:

https://bitbucket.org/xemacs/xemacs/commits/319e18d08654/
Changeset:   319e18d08654
User:        kehoea
Date:        2014-01-16 21:26:27
Summary:     Increase READ_BUF_SIZE, used in #'insert-file-contents-internal, to #x10000

src/ChangeLog addition:

2014-01-16  Aidan Kehoe  <[email protected]>

	* fileio.c (READ_BUF_SIZE):
	Stack sizes > 2**16 elicited bugs back in August 1996, but there's
	nothing to indicate this is still the case. GNU uses a block size
	of #x10000, and that size works better with the coding system
	buffering, improving performance there; move to this value.
Affected #:  2 files

diff -r ccaa851ae712e870ceb65d19226426d40d8de48c -r 319e18d08654e2b6b6a20bc496349fcc959ed69f src/ChangeLog
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,11 @@
+2014-01-16  Aidan Kehoe  <[email protected]>
+
+	* fileio.c (READ_BUF_SIZE):
+	Stack sizes > 2**16 elicited bugs back in August 1996, but there's
+	nothing to indicate this is still the case. GNU uses a block size
+	of #x10000, and that size works better with the coding system
+	buffering, improving performance there; move to this value.
+
 2014-01-16  Aidan Kehoe  <[email protected]>
 
 	* file-coding.c:

diff -r ccaa851ae712e870ceb65d19226426d40d8de48c -r 319e18d08654e2b6b6a20bc496349fcc959ed69f src/fileio.c
--- a/src/fileio.c
+++ b/src/fileio.c
@@ -2870,9 +2870,7 @@
 }
 
 
-/* Stack sizes > 2**16 is a good way to elicit compiler bugs */
-/* #define READ_BUF_SIZE (2 << 16) */
-#define READ_BUF_SIZE (1 << 15)
+#define READ_BUF_SIZE (2 << 16)
 
 DEFUN ("insert-file-contents-internal", Finsert_file_contents_internal,
        1, 7, 0, /*

Repository URL: https://bitbucket.org/xemacs/xemacs/

--

This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.