comparison src/ChangeLog @ 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 cde1608596d0
children 17c381a2f377
comparison
equal deleted inserted replaced
5306:cde1608596d0 5307:c096d8051f89
1 2010-11-20 Aidan Kehoe <kehoea@parhasard.net>
2
3 * abbrev.c (Fexpand_abbrev):
4 * alloc.c:
5 * alloc.c (Fmake_list):
6 * alloc.c (Fmake_vector):
7 * alloc.c (Fmake_bit_vector):
8 * alloc.c (Fmake_byte_code):
9 * alloc.c (Fmake_string):
10 * alloc.c (vars_of_alloc):
11 * bytecode.c (UNUSED):
12 * bytecode.c (Fbyte_code):
13 * chartab.c (decode_char_table_range):
14 * cmds.c (Fself_insert_command):
15 * data.c (check_integer_range):
16 * data.c (Fnatnump):
17 * data.c (Fnonnegativep):
18 * data.c (Fstring_to_number):
19 * elhash.c (hash_table_size_validate):
20 * elhash.c (decode_hash_table_size):
21 * eval.c (Fbacktrace_frame):
22 * event-stream.c (lisp_number_to_milliseconds):
23 * event-stream.c (Faccept_process_output):
24 * event-stream.c (Frecent_keys):
25 * event-stream.c (Fdispatch_event):
26 * events.c (Fmake_event):
27 * events.c (Fevent_timestamp):
28 * events.c (Fevent_timestamp_lessp):
29 * events.h:
30 * events.h (struct command_builder):
31 * file-coding.c (gzip_putprop):
32 * fns.c:
33 * fns.c (check_sequence_range):
34 * fns.c (Frandom):
35 * fns.c (Fnthcdr):
36 * fns.c (Flast):
37 * fns.c (Fnbutlast):
38 * fns.c (Fbutlast):
39 * fns.c (Fmember):
40 * fns.c (Ffill):
41 * fns.c (Freduce):
42 * fns.c (replace_string_range_1):
43 * fns.c (Freplace):
44 * font-mgr.c (Ffc_pattern_get):
45 * frame-msw.c (msprinter_set_frame_properties):
46 * glyphs.c (check_valid_xbm_inline):
47 * indent.c (Fmove_to_column):
48 * intl-win32.c (mswindows_multibyte_to_unicode_putprop):
49 * lisp.h:
50 * lisp.h (ARRAY_DIMENSION_LIMIT):
51 * lread.c (decode_mode_1):
52 * mule-ccl.c (ccl_get_compiled_code):
53 * number.h:
54 * process-unix.c (unix_open_multicast_group):
55 * process.c (Fset_process_window_size):
56 * profile.c (Fstart_profiling):
57 * unicode.c (Funicode_to_char):
58 Change NATNUMP to return 1 for positive bignums; changes uses of
59 it and of CHECK_NATNUM appropriately, usually by checking for an
60 integer in an appropriate range.
61 Add array-dimension-limit and use it in #'make-vector,
62 #'make-string. Add array-total-size-limit, array-rank-limit while
63 we're at it, for the sake of any Common Lisp-oriented code that
64 uses these limits.
65 Rename check_int_range to check_integer_range, have it take
66 Lisp_Objects (and thus bignums) instead.
67 Remove bignum_butlast(), just set int_n to an appropriately large
68 integer if N is a bignum.
69 Accept bignums in check_sequence_range(), change the functions
70 that use check_sequence_range() appropriately.
71 Move the definition of NATNUMP() to number.h; document why it's a
72 reasonable name, contradicting an old comment.
73
1 2010-11-17 Aidan Kehoe <kehoea@parhasard.net> 74 2010-11-17 Aidan Kehoe <kehoea@parhasard.net>
2 75
3 * fns.c (bignum_butlast): New. 76 * fns.c (bignum_butlast): New.
4 (Fnbutlast, Fbutlast): Use it. 77 (Fnbutlast, Fbutlast): Use it.
5 In #'butlast and #'nbutlast, if N is a bignum, we should always 78 In #'butlast and #'nbutlast, if N is a bignum, we should always