Mercurial > hg > xemacs-beta
annotate src/strcat.c @ 5307:c096d8051f89
Have NATNUMP give t for positive bignums; check limits appropriately.
src/ChangeLog addition:
2010-11-20 Aidan Kehoe <kehoea@parhasard.net>
* abbrev.c (Fexpand_abbrev):
* alloc.c:
* alloc.c (Fmake_list):
* alloc.c (Fmake_vector):
* alloc.c (Fmake_bit_vector):
* alloc.c (Fmake_byte_code):
* alloc.c (Fmake_string):
* alloc.c (vars_of_alloc):
* bytecode.c (UNUSED):
* bytecode.c (Fbyte_code):
* chartab.c (decode_char_table_range):
* cmds.c (Fself_insert_command):
* data.c (check_integer_range):
* data.c (Fnatnump):
* data.c (Fnonnegativep):
* data.c (Fstring_to_number):
* elhash.c (hash_table_size_validate):
* elhash.c (decode_hash_table_size):
* eval.c (Fbacktrace_frame):
* event-stream.c (lisp_number_to_milliseconds):
* event-stream.c (Faccept_process_output):
* event-stream.c (Frecent_keys):
* event-stream.c (Fdispatch_event):
* events.c (Fmake_event):
* events.c (Fevent_timestamp):
* events.c (Fevent_timestamp_lessp):
* events.h:
* events.h (struct command_builder):
* file-coding.c (gzip_putprop):
* fns.c:
* fns.c (check_sequence_range):
* fns.c (Frandom):
* fns.c (Fnthcdr):
* fns.c (Flast):
* fns.c (Fnbutlast):
* fns.c (Fbutlast):
* fns.c (Fmember):
* fns.c (Ffill):
* fns.c (Freduce):
* fns.c (replace_string_range_1):
* fns.c (Freplace):
* font-mgr.c (Ffc_pattern_get):
* frame-msw.c (msprinter_set_frame_properties):
* glyphs.c (check_valid_xbm_inline):
* indent.c (Fmove_to_column):
* intl-win32.c (mswindows_multibyte_to_unicode_putprop):
* lisp.h:
* lisp.h (ARRAY_DIMENSION_LIMIT):
* lread.c (decode_mode_1):
* mule-ccl.c (ccl_get_compiled_code):
* number.h:
* process-unix.c (unix_open_multicast_group):
* process.c (Fset_process_window_size):
* profile.c (Fstart_profiling):
* unicode.c (Funicode_to_char):
Change NATNUMP to return 1 for positive bignums; changes uses of
it and of CHECK_NATNUM appropriately, usually by checking for an
integer in an appropriate range.
Add array-dimension-limit and use it in #'make-vector,
#'make-string. Add array-total-size-limit, array-rank-limit while
we're at it, for the sake of any Common Lisp-oriented code that
uses these limits.
Rename check_int_range to check_integer_range, have it take
Lisp_Objects (and thus bignums) instead.
Remove bignum_butlast(), just set int_n to an appropriately large
integer if N is a bignum.
Accept bignums in check_sequence_range(), change the functions
that use check_sequence_range() appropriately.
Move the definition of NATNUMP() to number.h; document why it's a
reasonable name, contradicting an old comment.
tests/ChangeLog addition:
2010-11-20 Aidan Kehoe <kehoea@parhasard.net>
* automated/lisp-tests.el:
* automated/lisp-tests.el (featurep):
* automated/lisp-tests.el (wrong-type-argument):
* automated/mule-tests.el (featurep):
Check for args-out-of-range errors instead of wrong-type-argument
errors in various places when code is handed a large bignum
instead of a fixnum.
Also check for the wrong-type-argument errors when giving the same
code a non-integer value.
| author | Aidan Kehoe <kehoea@parhasard.net> |
|---|---|
| date | Sat, 20 Nov 2010 16:49:11 +0000 |
| parents | abe6d1db359e |
| children | 2aa9cd456ae7 |
| rev | line source |
|---|---|
| 428 | 1 /* Copyright (C) 1991 Free Software Foundation, Inc. |
| 2 This file is part of the GNU C Library. | |
| 3 | |
| 4 The GNU C Library is free software; you can redistribute it and/or | |
| 5 modify it under the terms of the GNU Library General Public License as | |
| 6 published by the Free Software Foundation; either version 2 of the | |
| 7 License, or (at your option) any later version. | |
| 8 | |
| 9 The GNU C Library is distributed in the hope that it will be useful, | |
| 10 but WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 11 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | |
| 12 Library General Public License for more details. | |
| 13 | |
| 14 You should have received a copy of the GNU Library General Public | |
| 15 License along with the GNU C Library; see the file COPYING.LIB. If | |
| 16 not, write to the Free Software Foundation, Inc., 675 Mass Ave, | |
| 17 Cambridge, MA 02139, USA. */ | |
| 18 | |
| 19 /* Synched up with: Not in FSF. */ | |
| 20 | |
| 21 # include <config.h> | |
| 22 # ifndef REGISTER /* Strictly enforced in 20.3 */ | |
| 23 # define REGISTER | |
| 24 # endif | |
| 25 | |
| 26 /* In HPUX 10 the strcat function references memory past the last byte of | |
| 27 the string! This will core dump if the memory following the last byte is | |
| 28 not mapped. | |
| 29 | |
| 442 | 30 Here is a correct version from, glibc 1.09. |
| 428 | 31 */ |
| 32 | |
| 33 char *strcat (char *dest, const char *src); | |
| 34 | |
| 35 /* Append SRC on the end of DEST. */ | |
| 36 char * | |
| 37 strcat (char *dest, const char *src) | |
| 38 { | |
| 39 REGISTER char *s1 = dest; | |
| 442 | 40 REGISTER const char *s2 = src; |
| 428 | 41 char c; |
| 42 | |
| 43 /* Find the end of the string. */ | |
| 44 do | |
| 45 c = *s1++; | |
| 46 while (c != '\0'); | |
| 47 | |
| 48 /* Make S1 point before the next character, so we can increment | |
| 49 it while memory is read (wins on pipelined cpus). */ | |
| 50 s1 -= 2; | |
| 51 | |
| 52 do | |
| 53 { | |
| 54 c = *s2++; | |
| 55 *++s1 = c; | |
| 56 } | |
| 57 while (c != '\0'); | |
| 58 | |
| 59 return dest; | |
| 60 } |