Github/emacs
2
0
Fork 1
mirror of git://git.sv.gnu.org/emacs.git synced 2026-06-14 12:31:25 +00:00
Code Issues Projects Releases Packages Wiki Activity

Simplify module_extract_big_integer size calcs

Browse source 
* src/emacs-module.c (module_bignum_count_max):
Now of type ptrdiff_t, instead of likely being of type size_t.
(module_extract_big_integer): Omit now-unnecessary prefix +, a
now-unnecessary eassert against PTRDIFF_MAX, and and an
unnecessary cast to ptrdiff_t.
This commit is contained in:
Paul Eggert 2026-05-17 22:31:26 -07:00
parent b9e20e3995
commit 7587bb2654
1 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/emacs-module.c
View file

@ -1046,7 +1046,7 @@ import/export overhead on most platforms.
/* Documented maximum count of magnitude elements. */
#define module_bignum_count_max \
((ptrdiff_t) min (SIZE_MAX, PTRDIFF_MAX) / sizeof (emacs_limb_t))
((ptrdiff_t) (min (SIZE_MAX, PTRDIFF_MAX) / sizeof (emacs_limb_t)))
/* Verify that emacs_limb_t indeed has unique object
representations. */
@ -1100,7 +1100,7 @@ module_extract_big_integer (emacs_env *env, emacs_value arg, int *sign,
suffice. */
EMACS_UINT u;
enum { required = (sizeof u + size - 1) / size };
static_assert (0 < required && +required <= module_bignum_count_max);
static_assert (0 < required && required <= module_bignum_count_max);
if (magnitude == NULL)
{
*count = required;
@ -1132,9 +1132,8 @@ module_extract_big_integer (emacs_env *env, emacs_value arg, int *sign,
return true;
}
size_t required_size = (mpz_sizeinbase (*x, 2) + numb - 1) / numb;
eassert (required_size <= PTRDIFF_MAX);
ptrdiff_t required = (ptrdiff_t) required_size;
eassert (required <= module_bignum_count_max);
eassert (required_size <= module_bignum_count_max);
ptrdiff_t required = required_size;
if (magnitude == NULL)
{
*count = required;