[PATCH] Fix memory leak (#1772) when importing modules more than once

Hi all, Attached is a straightforward fix for #1772. I put it on the milestone 5.4, but since it's such a simple fix, I think we could include it in 5.3 anyway. Cheers, Peter From 70e45fe529d51421fbeea8828dc1704e3a821d40 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Thu, 22 Jul 2021 13:22:00

[PATCH] Remove severely outdated README section

Sep 17 00:00:00 2001 From: Peter Bex Date: Wed, 21 Jul 2021 11:35:18 +0200 Subject: [PATCH] Remove severely outdated README section on compatibility This mentioned only the upgrade from 3 to 4, and had nothing about 4 to 5 anyway. Better to remove it completely to avoid confusion. --- README | 13

[PATCH] Fix #1727 with patch from ticket

. Likewise, quotes inside paths are illegal, therefore no quotes should be used inside PATH. It appears that the Windows shell cannot deal with quoting inside an argument, only when quotes surround the argument. This is not the case on Unix environments at all. Closes #1727 Signed-off-by: Peter Bex

[PATCH] Fix #1757 by not merging module environment into the syntactic environments of reexported macros

work if we keep it referring to only local macros, and I think we shouldn't mess too much with the environments where that's not needed, so I think it's fine. Cheers, Peter From 96827b77554e8b56838b0e47363326011b6e Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Wed, 14 Jul 2021 12:18:56 +02

[PATCH] Another fix for irregex to bring us to 0.9.10

Hi all, Here's one more patch to bring Irregex completely up to date with the just released 0.9.10 version. It fixes an issue where "bol" would overlap with newline characters in a weird way. Cheers, Peter From efe932f4fa7afbc56865d33edfbf6836c34ce919 Mon Sep 17 00:00:00 2001 From:

Re: [PATCH] show more helpful error message with "-l"

On Thu, Jul 01, 2021 at 10:44:22PM +0200, felix.winkelm...@bevuta.com wrote: > See patch, suggested by jcroisant on #chicken. Since -lxxx may > be ambiguous, not all cases may be caught, but it can still be helpful to new > users. Thanks, applied. The patch has some weird indentation and also

[PATCH] Fix for incorrect NFA compilation of "or" with empty sequence [was: Re: [PATCH] Fix for irregex incorrect matches]

On Mon, Jul 05, 2021 at 11:45:36AM +0200, Peter Bex wrote: > Hi there, > > The attached patch is a port of the most recent commit in the irregex > repository, which fixes this upstream ticket: > https://github.com/ashinn/irregex/issues/27 And here is another patch to fix a differe

[PATCH] Fix for irregex incorrect matches

Hi there, The attached patch is a port of the most recent commit in the irregex repository, which fixes this upstream ticket: https://github.com/ashinn/irregex/issues/27 Cheers, Peter From b552052f4085e84d662f70bb76cb4abf41ab25bc Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Mon, 5 Jul 2021 11

[PATCH] Add runtime option to fix randomization seed on startup

nd perhaps a good first stab at reducing the noisiness of our benchmarks. The patch should speak for itself. Cheers, Peter From eecb550157a0bf809132329928b9338c37875bd8 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Thu, 17 Jun 2021 13:28:57 +0200 Subject: [PATCH] Add new -:R runtime option to inf

[PATCH] Fix #1756 by replacing rest ops for explicitly consed rest args

ed change, hopefully the way the commits are structured should make it easy to review. Cheers, Peter From bf48a7f9dbb26e3461d1090da63ca9496d902287 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Wed, 16 Jun 2021 08:32:28 +0200 Subject: [PATCH 1/3] Add information to the db for rest args which ar

[PATCH] Fix #1675 by using theoretical maximum addressable range on 64-bit platforms

d6cc3d958c6f73a1bc7adaa89a3c71c7d0394b11 Mon Sep 17 00:00:00 2001 From: dieggsy Date: Sat, 10 Apr 2021 13:36:18 -0400 Subject: [PATCH] Increase maximum heap size on 64-bit machines Signed-off-by: Peter Bex --- NEWS | 3 +++ runtime.c | 3 ++- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/NEWS b

[PATCH] Partial fix for #1685

7 00:00:00 2001 From: Peter Bex Date: Thu, 20 May 2021 09:57:17 +0200 Subject: [PATCH] Partial fix for #1685 - quote shell variables on UNIX platforms This allows having shell metacharacters (like spaces) in path names. This does not yet fix the situation for Windows - there, quotation is context

[PATCH] Update irregex to 0.9.9

p-level-domain. Cheers, Peter From 1a7ee0d480b296d3a93f81d74854a2d899ed67f2 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Fri, 14 May 2021 08:32:48 +0200 Subject: [PATCH] Update irregex to upstream 0.9.9 (rev 3c367082) This fixes an edge case from upstream bug #24 where replacing empty strings

Re: [PATCH] make chicken-install cache depend on PROGRAM_PREFIX/SUFFIX

On Thu, Apr 15, 2021 at 11:05:36AM +0200, felix.winkelm...@bevuta.com wrote: > A patch by alicemaz, see #1713. Thanks, pushed. Cheers, Peter signature.asc Description: PGP signature

Re: [PATCH] make chicken-install cache depend on PROGRAM_PREFIX/SUFFIX

On Thu, Apr 15, 2021 at 01:47:07PM +, Diego wrote: > I wonder if it makes more sense to cache by (chicken-version) instead. For > example, is there a reason that two chicken installs with different > prefix/suffix shouldn't share the cache if their version were the same? They may have

[PATCH] Fix r7rs egg dependency list

Hi all, The fix for #1185 (topological-sort) seems to have caused quite a regression on Salmonella. See [1]; there are 22 eggs broken. Most of those seem to have a dependency on r7rs, which is broken. It looks like the fix for topological-sort simply exposed bugs in existing eggs because

Re: [PATCH] Remove another keyword? check [was: Re: [PATCH] Small cleanup to get rid of a few compilation warnings]

On Sun, Apr 11, 2021 at 01:13:17PM +0200, Peter Bex wrote: > Here's another trivial one in the same vein. And another one (I think this is the last one). Cheers, Peter From b0157017c6060a57d6b2fc87752446bc518cee57 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Mon, 12 Apr 2021 09:22:40 +0

Re: [PATCH] RISC-V machine identifier

On Sun, Apr 11, 2021 at 05:14:03PM +0200, felix.winkelm...@bevuta.com wrote: > Hi! > > The attached patch sets the machine type identifier on RISC-V properly. > Thanks to "Irvine" for providing this information. Thanks, pushed. Cheers, Peter signature.asc Description: PGP signature

[PATCH] Remove another keyword? check [was: Re: [PATCH] Small cleanup to get rid of a few compilation warnings]

On Sat, Apr 10, 2021 at 03:25:51PM +0200, Peter Bex wrote: > Hi all, > > Here's a simple patch. Found while reviewing Megane's patch for > improving line number reporting in [ei]r-macro-transformer; this > recompiled expand.scm which rubbed those compiler warnings in my > fac

[PATCH] Clean up define-specialization macro, get rid of ##sys#globalize

e environment too). I think this is a nice simplicification and another small but useful step towards whittling down ##sys#alias-global-hook. Cheers, Peter From 90837acd09ebfc2c6dda6874084c6bd365ef72e5 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Sat, 10 Apr 2021 20:11:40 +0200 Subject: [PATCH 1/2]

[PATCH] Small cleanup to get rid of a few compilation warnings

: Peter Bex Date: Sat, 10 Apr 2021 15:20:04 +0200 Subject: [PATCH] Minor cleanup Symbols and keywords are disjoint now, so we can remove the needless checks that say (and (symbol? x) (not (keyword? x))), because if something's a symbol, it cannot be a keyword too. This gets rid of some

Re: [PATCH] Make ir-macro-transformer retain more of line-number information

On Sat, Apr 10, 2021 at 10:07:53AM +0300, megane wrote: > Hi, > > a small patch to make the compiler maintain more fine grained line number > info. Thanks, pushed with a small change: I lifted the let* of "cur" up into the and-let* above, because "cur" is always truthy anyway, and it makes no

Re: [PATCH] handle eof in read-chunks

On Thu, Apr 01, 2021 at 09:46:45PM +0200, Mario Domenech Goulart wrote: > Hi, > > Attached are patches which hopefully fix #1744 [0]. I haven't been able > to reproduce the issue on my system, but I believe the current code is > subject to the problem whose results we see in salmonella logs. I

[PATCH] Speed up srfi-69 by caching current min and max length based on load and *actually resizing*

Hi all, I'm sure you've all read Ben Hoyt's blog post from earlier this week, https://benhoyt.com/writings/count-words/ Of course (duh) I tried to implement a version in CHICKEN. So did Vasilij and Mario, and we all came to the conclusion that our performance on that simple task is abysmal, we

[PATCH] Fix #1730

Hi all, Here's a straightforward patch for #1730. Cheers, Peter From 6b70c5d0e13c0f88d67aa2eb508b327b70bd4aa3 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Wed, 3 Mar 2021 07:48:46 +0100 Subject: [PATCH] Fix numerator and denominator error handling (fixes #1730) --- NEWS| 2

[PATCH] Fix debug output of heap and stack size

Hi all, Here's a simple fix for the heap size printing, which will say "heap resized to 0 bytes" if you set it to something large enough, like 4G. Cheers, Peter From a04b59d38e412af1d2860283f6c61e9acbd9c3f6 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Fri, 15 Jan 2021 12:04:30 +01

[PATCH] Fix #1725 by adding optional argument for scheme#log to types.db

Hi all, Here's a fix for a small oversight in the types database. Cheers, Peter From a7030e12c709f9524636e2a414a70228c5484d89 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Wed, 13 Jan 2021 08:43:19 +0100 Subject: [PATCH] Add second optional argument for scheme#log to types database. Also add

Re: [PATCH] a trivial optimization

On Sat, Nov 14, 2020 at 01:14:46PM +0100, felix.winkelm...@bevuta.com wrote: > Hi! > > The attached (tiny) patch removes setting the fixnum-mark bit > after binary OR - fixnums have the bit set, anyway. Doesn't make > much of a difference, but for completeness' sake... Thanks, pushed. Cheers,

[PATCH] Fix #1703 by avoiding re-use of argvector in CPS calls that refer to original argvector in the CPS call's arguments

Hi all, Here's a fix for #1703. The commit message says it all. Cheers, Peter From 20c2cc89d9e45faf3ec51c30b260c62bc2fdc9ce Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Wed, 16 Sep 2020 13:02:46 +0200 Subject: [PATCH] Don't re-use argvector when the CPS call's arguments use rest-ops (#1703

Re: [PATCH] FIx bug in csi's ",c" command

On Tue, Jul 07, 2020 at 09:13:49AM +0200, felix.winkelm...@bevuta.com wrote: > Hi! > > This patch fixes a problem with the ",c" toplevel csi command when > the last exception was not a normal "property" condition. So, e.g. > something like > > #;> (signal 1) > #;> ,c > > would trigger an error

[PATCH] Update irregex to 0.9.8 for issue in irregex-replace/all with positive lookbehind

Hi all, Here's a patch to bring our core copy of irregex up to date with the upstream change Alex pushed yesterday to fix the issue reported on chicken-users by Kay Rhodes. Cheers, Peter From 14536ad80fee5c43471da21d4e5336b4317819ad Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Tue, 14 Jul

Re: [PATCH] Clean up some hacks and obsolete code

On Thu, May 21, 2020 at 03:12:33PM -0500, Jim Ursetto wrote: > Hey Peter, Hi Jim, Good to see you're back! > I might be misunderstanding, but quite a few Chicken 5 eggs still use > define-record-printer, and it seems to work fine in Chicken 5.1.0 and 5.2.0. hm, maybe it's just the way we used

Re: [PATCH] Clean up some hacks and obsolete code

On Wed, May 20, 2020 at 08:45:51PM +0200, Peter Bex wrote: > Regarding patch 0002, it looks like we forgot to change > define-record-printer (perhaps because it, itself is deprecated) > to use set-record-printer! instead of ##sys#register-record-printer, > which means we

[PATCH] Clean up some hacks and obsolete code

to change define-record-printer (perhaps because it, itself is deprecated) to use set-record-printer! instead of ##sys#register-record-printer, which means we can't remove that definition just yet. Cheers, Peter From 518b25f881c6c15626487fc768895d66cbb13a75 Mon Sep 17 00:00:00 2001 From: Peter Bex

Re: Fwd: Comments on draft #3

On Sun, May 03, 2020 at 12:55:40PM -0400, John Cowan wrote: > This is the first of two patches that allow macros to be expanded when they > appear in operand position, so that if pi is a macro for 3.14159, then (+ 5 > pi) will be expanded to (+ 5 3.14159). This allows macros that work like >

Re: More millisecond procedures

On Sun, May 03, 2020 at 03:48:46PM +0300, Lassi Kortela wrote: > > Attached is a set of two patches. The first one simply adds a > > deprecation notice to current-milliseconds and adds the new procedure > > current-process-milliseconds. > > Racket also has `current-inexact-milliseconds`:

[PATCH] Deprecate current-milliseconds in favor of current-process-milliseconds and improve behaviour a bit [was: Re: Exposing subsecond precision in current-seconds]

to define current-process-milliseconds as time since process startup, and drop the wishy-washy documentation that it's time "since process or system startup". Cheers, Peter From a53e6a707e34fa26f42a110b8948ef9710c0d86e Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Sun, 3 May 2020 12:15

Re: Exposing subsecond precision in current-seconds

On Thu, Apr 30, 2020 at 01:04:08AM +0200, felix.winkelm...@bevuta.com wrote: > > I was trying to illustrate how annoying it'd be to have to jump through > > hoops if you need to print the time in a human-readable way or convert > > it to a more "standard" epoch-based representation that includes >

Re: Exposing subsecond precision in current-seconds

On Wed, Apr 29, 2020 at 10:40:06PM +0200, felix.winkelm...@bevuta.com wrote: > > On Wed, Apr 29, 2020 at 06:21:12PM +0200, felix.winkelm...@bevuta.com wrote: > > > > On Wed, Apr 29, 2020 at 05:36:10PM +0200, felix.winkelm...@bevuta.com > > > > wrote: > > > > > Microseconds since epoch? Are you

Re: Exposing subsecond precision in current-seconds

On Wed, Apr 29, 2020 at 06:21:12PM +0200, felix.winkelm...@bevuta.com wrote: > > On Wed, Apr 29, 2020 at 05:36:10PM +0200, felix.winkelm...@bevuta.com wrote: > > > Microseconds since epoch? Are you sure about this? > > > > What else would the new current-microseconds return? > > Since startup, or

Re: Exposing subsecond precision in current-seconds

On Wed, Apr 29, 2020 at 05:36:10PM +0200, felix.winkelm...@bevuta.com wrote: > > One thing I'm concerned about is that there's current-milliseconds, which > > returns the milliseconds since startup, while we'd have current-seconds > > and current-microseconds which return the (micro)seconds since

Re: Exposing subsecond precision in current-seconds

On Wed, Apr 29, 2020 at 06:38:54PM +1200, Evan Hanson wrote: > On 2020-04-27 21:26, felix.winkelm...@bevuta.com wrote: > > > - Add a new procedure in `(chicken time)` or `(chicken time posix)` that > > > gives > > > you both seconds and nanoseconds, be it as values, list, vector, ... > > >

Re: [PATCH] fix for #1689 (argc check with -O0)

On Sun, Apr 19, 2020 at 06:54:31PM +0200, felix.winkelm...@bevuta.com wrote: > My attempt to use the "potential-values" db entry doesn't seem to > lead to anything, all I can come up with in the moment is this lame > attempt to fix the situation, i.e. by only signalling an error for > non-global

[PATCH] A few simplifications/maintainability improvements of the GC

/simplify-gc/clang/freebsd/x86-64/2020/04/18/salmonella-report/ And I also ran the benchmarks against this, and the performance stays the same (modulo some noise differences, as usual). Cheers, Peter From 7f8cc0e0a49edf21784acd2f5aa21b5cda91f083 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Tue

Re: [PATCH 0/1] Prevent excessive major gcs by preserving a decent amount of unused heap

On Mon, Apr 13, 2020 at 02:13:24PM +0200, Peter Bex wrote: > On Mon, Apr 13, 2020 at 11:51:41AM +1200, Evan Hanson wrote: > > On 2020-04-12 19:11, Peter Bex wrote: > > > What's the deal with the FAIL for knucleotide in both patched versions? > > > > Oh, that's bec

Re: [PATCH 0/1] Prevent excessive major gcs by preserving a decent amount of unused heap

On Mon, Apr 13, 2020 at 11:51:41AM +1200, Evan Hanson wrote: > On 2020-04-12 19:11, Peter Bex wrote: > > What's the deal with the FAIL for knucleotide in both patched versions? > > Oh, that's because knucleotide needs srfi-69 and I didn't install it for > the testing builds. Ah

Re: [PATCH] fix for #1689 (argc check with -O0)

On Tue, Apr 07, 2020 at 12:48:12PM +0200, felix.winkelm...@bevuta.com wrote: > Attached is a patch to address #1689 by moving the argc check for a known > procedure call into the analysis phase (on first analysis) so that even with > -O0 > this check is done, as runtime-checks are disabled for

Re: [PATCH] fix for #1689 (argc check with -O0)

On Tue, Apr 07, 2020 at 12:48:12PM +0200, felix.winkelm...@bevuta.com wrote: > Attached is a patch to address #1689 by moving the argc check for a known > procedure call into the analysis phase (on first analysis) so that even with > -O0 > this check is done, as runtime-checks are disabled for

Re: [PATCH 0/1] Prevent excessive major gcs by preserving a decent amount of unused heap

On Sun, Mar 29, 2020 at 10:10:13AM +1300, Evan Hanson wrote: > Hi there, > > I've played around with this first patch a bit, and I think I've found a > way to apply the same technique without complicating the logic in > C_reclaim(). Attached is a sign-off of megane's patch that introduces > the

Re: [PATCH] fix for #1689 (argc check with -O0)

On Tue, Apr 07, 2020 at 12:48:12PM +0200, felix.winkelm...@bevuta.com wrote: > Attached is a patch to address #1689 by moving the argc check for a known > procedure call into the analysis phase (on first analysis) so that even with > -O0 > this check is done, as runtime-checks are disabled for

Re: irregex-match? & specialize question

On Tue, Mar 24, 2020 at 10:46:36AM -0700, Kon Lovett wrote: > = I must be missing something Hi Kon, This is an interesting little bug you found here. I don't understand it at all. However, I tested with older CHICKEN versions and it seems to break with 4.9.0 up till HEAD, so it's not a new

Re: [PATCH] Fix #1665 by blocking direct calls to externally inlineable procedures

On Wed, Feb 12, 2020 at 06:18:51PM +0200, megane wrote: > Is there any technical reason we couldn't call external functions > directly? In principle, this should be possible: 1) Just mark it "extern" in its own compilation unit 2) With the external inline file, also include a corresponding

[PATCH] Fix #1665 by blocking direct calls to externally inlineable procedures

, but the (bar 'zzz) call does not. I don't think the unroll limit is supposed to apply to the different calls before replacement. Cheers, Peter From aa3447ac747611ca6b381f398e65c051fd79f761 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Mon, 10 Feb 2020 15:54:26 +0100 Subject: [PATCH] Don't directly call

Re: [patch] csi needs flonum arithmetic

On Mon, Feb 03, 2020 at 09:30:56PM +0100, ko...@upyum.com wrote: > felix.winkelm...@bevuta.com wrote: > > This patch removes the "fixnum-arithmetic" declaration from csi.scm, > > since "report" (,r) crashes due to flonums in the symbol-table statistics > > vector. > > Applied. Also cherry-picked

[PATCH] Fix compilation issue in 5.2.0rc1

fixes the issue. His code now compiles and also works as expected. I think the solution is a bit hacky, but I don't know of a better way to do this. Suggestions welcome! Cheers, Peter From 8b20b9adffa556b18d1a8a805754094ffe53ea77 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Fri, 17 Jan 2020

[PATCH] Update irregex to 0.9.7

process. Cheers, Peter From 3de0366cb1a098ab205f2003b824107447d6e9e6 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Thu, 2 Jan 2020 20:46:44 +0100 Subject: [PATCH] Update irregex to the 0.9.7 release (upstream commit 353b8db8) This makes the behaviour of irregex-fold and irregex-split more in line

[PATCH] Fix #1661 by updating irregex to upstream

Sep 17 00:00:00 2001 From: Peter Bex Date: Mon, 30 Dec 2019 18:31:35 +0100 Subject: [PATCH] Update irregex to latest upstream (b194cab) to fix #1661 irregex-replace and irregex-replace/all now work properly for empty matches: they don't drop characters and will actually insert the replacement

Re: Time to make a 5.2.0 release candidate?

On Sun, Dec 29, 2019 at 01:56:02AM +0100, felix.winkelm...@bevuta.com wrote: > It would of course be better to fix the eggs first, but this might take time, > especially since some maintainers may not react instantly. Agreed. > For the autodetection patch: As I understand it, giving an explicit

Time to make a 5.2.0 release candidate?

Hi all, I think we're about ready to start the process for a 5.2.0 release. What do you think? Should we wait for the eggs from #1655 to be fixed? What about the plaform auto-detection patch? Cheers, Peter signature.asc Description: PGP signature

Re: Patch for autodetection of platforms

f if you want to, but the README examples no longer mention it. Signed-off-by: Peter Bex --- GNUmakefile | 4 ++- Makefile.detect | 76 +++ NEWS | 3 ++ README| 39 ++ distribution/manif

Re: [PATCH] make -:hs0 truly disable heap shrinking

On Sun, Dec 22, 2019 at 11:09:12AM +0100, felix.winkelm...@bevuta.com wrote: > Hi! > > "-:hs0" is currently ineffective, since validity checks between parsing > the option and initializing the heap overwrite the setting. > > This patch initializes the heap-parameter variables statically, >

Re: [PATCH] Add redact-gensyms and output files to testclean target

On Wed, Dec 11, 2019 at 03:47:51PM +, Evan Hanson wrote: > Previously, the "testclean" target would leave a few files lying around: > the redact-gensyms executable that's used to iron out unique identifiers > in the tests, and the output files it produces. This commit just makes > sure those

Re: [PATCH] Fix #1658 and add testcases for rest args

On Sun, Dec 15, 2019 at 10:06:06PM +0100, felix.winkelm...@bevuta.com wrote: > I think you forgot the rest-arg-tests.scm file. Oops, you're right. Here it is. Cheers, Peter From af34cce1815a496a5bc98d83b88bebbe21f14f9b Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Sun, 15 Dec 2019 20:39

[PATCH] Fix #1658 and add testcases for rest args

valid after the procedure's inlined (the argvector would be that of the caller, not the callee). Cheers, Peter From aeeb5eabe464d9e2eda6e03b5ea5f36b1e40b856 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Sun, 15 Dec 2019 20:39:20 +0100 Subject: [PATCH] Fix remaining rest-argument bug #1658 After

[PATCH] Fix for qt-light (rest operation inside closure would fail)

From ab9d25d80b8ec0e751f130248637c5cbbbf233b1 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Wed, 11 Dec 2019 13:24:48 +0100 Subject: [PATCH] Fix restoration of rest operations inside closures When a rest operation would have to be undone due to no longer having access to the original procedure's

Re: [PATCH] Add help info to csi banner

On Sun, Dec 08, 2019 at 08:32:49AM +0200, megane wrote: > Hi, > > Here's a small helper for newbies (and those who can't remember, like > me). Thanks, good idea. I've updated the manual as well to include this in the banner. Cheers, Peter signature.asc Description: PGP signature

Re: [PATCH] Improve heap shrinkage calculation

On Sun, Dec 01, 2019 at 11:22:37PM +0100, felix.winkelm...@bevuta.com wrote: > It seems to work fine, but "-:hs0" doesn't seem to work. I tried > > csi -:d -:hs0 > > (define x (make-vector 1000)) > > (##sys#gc) > > (##sys#gc) > > ... > > The heap is still shrunk on each

[PATCH] Improve heap shrinkage calculation

Cheers, Peter From db141cfc0ac4e8ca37c823749c24e5acbe128e80 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Sun, 1 Dec 2019 15:36:36 +0100 Subject: [PATCH] Improve heap shrinkage factor. The old calculation was confused because it took a percentage of a percentage. This adds a new option to set heap shrinkag

[PATCH] Fix #1649 by using '* as return value type if foreign type has a retconv procedure

type in these sitations. On the other hand, to me it doesn't seem very useful to have conversion only in one direction, so I think for 5.2 it's enough to have the fix as attached. Cheers, Peter From 78cc27a8c0518a109645d094c68065d027571e1e Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Fri, 29

Re: [PATCH] Fix decompose-directory and change drives before attempting to build eggs on Windows

On Mon, Nov 11, 2019 at 08:57:11PM +0100, Peter Bex wrote: > I actually prefer it the "cd /d" way, because then we can't accidentally > forget to introduce this elsewhere: we can just call cd-command and it > will work on Windows if it works on UNIX. The code will be shorter a

[PATCH] Also optimize case-lambda using rest-cdr mechanism (#1623)

the "c" argument in the C code which indicates the length of the argvector instead of taking the length of the rest list variable. Cheers, Peter From 96b1d2584a622b6ef8a9ed65daad4acffb00d1df Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Thu, 21 Nov 2019 17:06:26 +0100 Subject: [PATCH] Add

[PATCH] Fix allocation size for C_s_a_i_digits_to_integer

ba1253332d42e1296513d29ac85d201fab9c8e46 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Sun, 17 Nov 2019 12:54:19 +0100 Subject: [PATCH] Fix allocation size for C_s_a_i_digits_to_integer Found by jjhoo using Valgrind: C_bignum2 needs 4 words (header, sign and the two digit words), plus the 2 words for the bignum wrapper

Re: [PATCH] Fix decompose-directory and change drives before attempting to build eggs on Windows

On Tue, Nov 12, 2019 at 08:27:30AM +1300, Evan Hanson wrote: > On 2019-11-11 13:57, Lassi Kortela wrote: > > > > The fix is to first switch drives, then run cd. > > > > `cd /d c:\foo` can be used to switch the drive and the directory both in one > > command. > > Oh, that's good to know, thanks

[PATCH] Fix decompose-directory and change drives before attempting to build eggs on Windows

decompose-pathname doesn't yield the drive letter. So for now I think the patch is fine as it is. It's still a bit awkward though. Cheers, Peter From 9a40e5f993d8e7051996f1a0974a3d3ad8e70ee0 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Sat, 9 Nov 2019 16:52:36 +0100 Subject: [PATCH 1/2]

Re: [PATCH] Pick rest argument values directly from the argvector without consing

nd.scm? I was unsure if the rest argument is always tN where N is the number of arguments. Cheers, Peter From 082b24633ddf831f63a77661b88d97b25fd08f2b Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Sat, 17 Aug 2019 13:30:09 +0200 Subject: [PATCH] Replace car/cdr/null? on rest args with d

Re: [PATCH] Pick rest argument values directly from the argvector without consing

On Sun, Oct 27, 2019 at 04:22:58PM +0100, Jörg F. Wittenberger wrote: > Likely. Though I've seen graphics code, which had a huge amount of keyword > args. Will this change speed them up too? No, keyword arguments are basically handled as plists. It might be possible to translate this to some

Re: [PATCH] Pick rest argument values directly from the argvector without consing

On Sun, Oct 27, 2019 at 01:16:10PM +0100, Peter Bex wrote: > This super simple benchmark runs 25% faster with -O3: > > (define (foo #!optional (a 1) (b 2)) > (+ a b)) > > (time > (let lp ((i 0)) > (unless (= i 1) > (foo 1 3) > (lp (add1

[PATCH] Pick rest argument values directly from the argvector without consing

: Peter Bex Date: Sat, 17 Aug 2019 13:30:09 +0200 Subject: [PATCH] Replace car/cdr/null? on rest args with direct argvector references where possible In the generated C code, we don't need C_build_rest() to dynamically build a list containing everything in the argvector beyond the named arguments

Re: [Chicken-hackers] [PATCH] check for exported types, constants, inline procedures (#1346)

. OTOH, this trick we're using in core might be used in other real world codebases as well... Cheers, Peter From d014872716a97628c96cf50095237828d643b856 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Thu, 10 Oct 2019 07:47:24 +0200 Subject: [PATCH] Check if there _is_ a value or syntax bin

[Chicken-hackers] [PATCH] Fix #1633 by detecting nonexistent field names (typos) in constructor argument list in define-record-type

Hi all, Here's a relatively straightforward patch for #1633; we simply walk the constructor's arguments and check that there's a matching field definition for it. If not, bail out with syntax-error. Cheers, Peter From 7606cb342a09abb3f0c5798136811f65e736339b Mon Sep 17 00:00:00 2001 From: Peter

Re: [Chicken-hackers] [PATCH] Fix #1620 by ignoring captured state of replaced variables

On Sun, Sep 15, 2019 at 01:16:50PM +0200, Peter Bex wrote: > On Sun, Aug 25, 2019 at 04:34:02PM +0200, Peter Bex wrote: > > On Sat, Jul 20, 2019 at 11:51:28AM +0300, megane wrote: > > > Here's a new patch that drops the (not captured) check. > > > > Thanks for maki

Re: [Chicken-hackers] [PATCH] check for exported types, constants, inline procedures (#1346)

On Sat, Oct 05, 2019 at 06:45:42PM +0200, felix.winkelm...@bevuta.com wrote: > Right, please just apply your patch, if you are ok with my changes. I agree > about the types. Done! Thanks for pushing forward on this. Cheers, Peter signature.asc Description: PGP signature

Re: [Chicken-hackers] [PATCH] check for exported types, constants, inline procedures (#1346)

nd export-identifier check When finalizing a module, ensure that exported identififiers do not refer to types, inline-procedures or constants. Signed-off-by: Peter Bex --- NEWS| 5 +++-- core.scm| 5 - modules.scm | 12 +++- 3 files changed, 14 insertions(+), 8 deletions

Re: [Chicken-hackers] [PATCH] Improve srfi-13 performance quite a bit by inlining optarg handling

On Sat, Sep 28, 2019 at 11:27:49AM -0700, Matt Welland wrote: > foof-loop I've just ported this to CHICKEN 5, from the original upstream sources. It should be available shortly. Cheers, Peter signature.asc Description: PGP signature ___

Re: [Chicken-hackers] call-cc.org push slowness

On Sun, Sep 29, 2019 at 12:08:06PM +0200, felix.winkelm...@bevuta.com wrote: > Hi! > > I noticed long pauses when pushing git commits or committing to svn. I just noticed the same. I think it was vandusen not responding on the post-commit hook. I restarted the bot, so I think the problem

[Chicken-hackers] [PATCH] Improve srfi-13 performance quite a bit by inlining optarg handling

Hi all, Attached is a relatively straightforward patch for SRFI-13. It changes the let-string-start+end macro (and also let-string-start+end2 but that isn't exported) so that it uses let-optionals* and the entire picking apart of the rest arg list into start/end can be completely inlined, as

Re: [Chicken-hackers] [PATCH] Revert half of "Add some optimizer simplification rules"

On Mon, Sep 16, 2019 at 12:11:13PM +0300, megane wrote: > Hi, > > This reverts the rewrite that unearthed #1648. Pushed! At least we'll have a sane Salmonella again. Cheers, Peter signature.asc Description: PGP signature ___ Chicken-hackers mailing

Re: [Chicken-hackers] [PATCH] Some simplification rules for nested ##core#inline forms

On Wed, Sep 04, 2019 at 11:59:31AM +0200, felix.winkelm...@bevuta.com wrote: > The attached patch adds two optimization rules for certain uses of > ##core#inline. It basically rewrites > > (let (( (##core#inline ...))) > ( ... ...)) > > into > > ( ... (##core##inline ...) ...) It looks like

Re: [Chicken-hackers] [PATCH] Detect self-reference in "letrec"

On Sun, Sep 15, 2019 at 05:12:00PM +0200, felix.winkelm...@bevuta.com wrote: > I moved the variable into c.-e. and also changed the error message > to be slightly more useful. Thanks, pushed! Cheers, Peter signature.asc Description: PGP signature ___

[Chicken-hackers] [PATCH] Fix how chicken-install handles server errors and add redirect support

sending the new request through the proxy. The first patch fixes these issues. While in there, I decided that adding support for redirects shouldn't be too hard, so that's what the second patch does. Cheers, Peter From da2b5d0cbe5b67ea120b5d9157a64ee2eb3fe164 Mon Sep 17 00:00:00 2001 From: Peter Bex

Re: [Chicken-hackers] [PATCH] Fix #1620 by ignoring captured state of replaced variables

On Sun, Aug 25, 2019 at 04:34:02PM +0200, Peter Bex wrote: > On Sat, Jul 20, 2019 at 11:51:28AM +0300, megane wrote: > > Here's a new patch that drops the (not captured) check. > > Thanks for making that! Now that my original patch has been pushed, > here's an incremental pa

Re: [Chicken-hackers] [PATCH] Some simplification rules for nested ##core#inline forms

On Wed, Sep 04, 2019 at 11:59:31AM +0200, felix.winkelm...@bevuta.com wrote: > The attached patch adds two optimization rules for certain uses of > ##core#inline. It basically rewrites > > (let (( (##core#inline ...))) > ( ... ...)) > > into > > ( ... (##core##inline ...) ...) > > plus a

Re: [Chicken-hackers] [PATCH] Detect self-reference in "letrec"

On Wed, Aug 28, 2019 at 01:41:22PM +0200, felix.winkelm...@bevuta.com wrote: > An attempt to fix #1645. See commit message for more details. Hi Felix, I've taken a look at it and the logic itself looks good, but I find the usage of fluid-let on a global variable unsettling. Why not pass the

Re: [Chicken-hackers] [PATCH 4/6] * scrutinizer.scm: Infer more exact types after set!

On Thu, Aug 22, 2019 at 02:51:26PM +0300, megane wrote: > Hi, > > I'm working on some inference improvements and I noticed the blist keeps > accumulating some bogus entries. Commit 0003 removes some of those. Hi Megane, I've pushed your cleanups because I believe they make the code and

Re: [Chicken-hackers] [PATCH] Fix couple of hangs related to finalizers and (gc #t)

On Sat, Aug 10, 2019 at 08:04:31AM +0300, megane wrote: > megane writes: > > Also signal an error if trying to re-enter. This also would cause an > > infinite loop (when calling (gc #t) inside a finalizer). It would > > need special handling if re-entry is to be supported. > > This is too

Re: [Chicken-hackers] [PATCH] Fix #1294 by mentioning in the docs that define-record-printer is not a definition

' macro, which isn't a "real" definition (see #1294). Signed-off-by: Peter Bex --- DEPRECATED| 3 +++ NEWS | 3 +++ chicken-syntax.scm| 2 +- chicken.base.import.scm | 2 ++ distributio

Re: [Chicken-hackers] [PATCH] Disallow exporting external variables

On Sun, Aug 25, 2019 at 12:33:50PM +0200, felix.winkelm...@bevuta.com wrote: > See #1346. Thanks, pushed. I'll keep the ticket open for define-constant and define-type. I'm not sure about the latter though, because types should be in a different namespace than identifiers (you can have a type x

Re: [Chicken-hackers] [PATCH] Fix #1620 by ignoring captured state of replaced variables

rom 03000b61cfd13bffe809c0908b2144f86a4f49f0 Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Sun, 25 Aug 2019 16:23:11 +0200 Subject: [PATCH] Also allow captured variables with known values from being replaced This should still be safe. Only when the variable is assigned to is this not allowed. This change sho

Re: [Chicken-hackers] [PATCH] Preserve environment when executing module-registration

On Sun, Aug 25, 2019 at 03:36:47PM +0200, felix.winkelm...@bevuta.com wrote: > This patch wraps uses of ##sys#register-compiuled-module in a form > that preserves the syntactic environment. Otherwise import ed identifers > leak into the global namespace. See #1548. Thanks, pushed! Cheers, Peter

[Chicken-hackers] [PATCH] Fix #1470 and unignore clean/pure declarations

Mon Sep 17 00:00:00 2001 From: Peter Bex Date: Sun, 25 Aug 2019 12:06:05 +0200 Subject: [PATCH 1/2] Also load "clean" and "pure" annotations from types.db when not specializing There seems to be no good reason for skipping just these two when loading types databases. All the o

<    1   2   3   4   5   6   7   8   9   10   >