[PATCH] add 'concat! : List % -> %' in URAGG

8 views
Skip to first unread message

oldk1331

unread,
Jul 23, 2019, 8:11:03 AM7/23/19
to fricas...@googlegroups.com
We already have 'concat : List % -> %', and it's equally
useful to have its destructive counterpart, for long lists,
this version can save many many copies.

Although 'concat : List % -> %' is exported in LNAGG,
I believe that the proper place for
'concat/concat! : List % -> %' should be in URAGG.
desctructive-concat-for-URAGG.patch

oldk1331

unread,
Jul 24, 2019, 8:57:28 AM7/24/19
to fricas...@googlegroups.com
There's a bug: it gives error when the first aggregate
in the list is empty, see my updated version.
desctructive-concat-for-URAGG-v2.patch

Waldek Hebisch

unread,
Jul 24, 2019, 9:45:41 AM7/24/19
to fricas...@googlegroups.com
oldk1331 wrote:
>
> There's a bug: it gives error when the first aggregate
> in the list is empty, see my updated version.

OK, please commit

--
Waldek Hebisch
Reply all
Reply to author
Forward
0 new messages