Re: [PATCH] Reimplement module functions

emacs-devel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Reimplement module functions

From:	Philipp Stephani
Subject:	Re: [PATCH] Reimplement module functions
Date:	Sun, 21 May 2017 20:20:15 +0000

Paul Eggert <address@hidden> schrieb am Sa., 20. Mai 2017 um 22:46 Uhr:

A couple of minor things I noticed in the recently-installed patch:

> const short minargs = function->min_arity;

In theory at least the arity might be greater than 32767, so this should be
ptrdiff_t.

Yes, the 'short' was a leftover from a previous version that used a misc type instead of a pseudovector. Thanks for fixing.

[Prev in Thread]

Current Thread

[Next in Thread]

[PATCH] Reimplement module functions, Philipp Stephani, 2017/05/13
- Re: [PATCH] Reimplement module functions, Eli Zaretskii, 2017/05/13
  - Re: [PATCH] Reimplement module functions, Noam Postavsky, 2017/05/13
    - Re: [PATCH] Reimplement module functions, Eli Zaretskii, 2017/05/14
  - Re: [PATCH] Reimplement module functions, Aurélien Aptel, 2017/05/14
    - Re: [PATCH] Reimplement module functions, Eli Zaretskii, 2017/05/14
  - Re: [PATCH] Reimplement module functions, Philipp Stephani, 2017/05/14
    - [PATCH] Reimplement module functions, Philipp Stephani, 2017/05/14
    - Re: [PATCH] Reimplement module functions, Philipp Stephani, 2017/05/20
    - Re: [PATCH] Reimplement module functions, Paul Eggert, 2017/05/20
    - Re: [PATCH] Reimplement module functions, Philipp Stephani <=

Prev by Date: Re: [PATCH 01/13] Nextstep: Use instancetype explicit return type
Next by Date: Re: [PATCH] Clean up a couple of compiler warnings
Previous by thread: Re: [PATCH] Reimplement module functions
Next by thread: Re: [elpa] master e2dd351: * packages/other-frame-window: bump version; improve minor mode doc string
Index(es):
- Date
- Thread