Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

iter-yield from lambda currently broken #136

Open
okamsn opened this issue Nov 9, 2022 · 0 comments
Open

iter-yield from lambda currently broken #136

okamsn opened this issue Nov 9, 2022 · 0 comments
Labels
bug Something isn't working enhancement New feature or request upstream Applies to a dependecy

Comments

@okamsn
Copy link
Owner

okamsn commented Nov 9, 2022

As of the time of writing, iter-yield doesn't work in lambdas or when sharp quoted inside of iter-defun or iter-lambda. As a consequence, iter-yield doesn't work with seq-do, seq-doseq, and other mapping functions and macros which expand to such.

As a consequence of that, iterators cannot currently be used to work around the limitations of seq-do, mapatoms, and other methods that we might wish to use to produce sequence elements without producing an intermediate list, as desired in #134 and #135.

This has been reported upstream by another Emacs user (https://debbugs.gnu.org/cgi/bugreport.cgi?bug=59140).

@okamsn okamsn added bug Something isn't working upstream Applies to a dependecy enhancement New feature or request labels Nov 9, 2022
okamsn added a commit that referenced this issue Dec 5, 2024
- Create command `seq`.
  - Remove `seq` as an alias for `sequence`.
  - Duplicate `sequence` tests for `seq`.
  - Add test `sequence-not-seq` to make sure we didn't confuse the two anywhere.

- Create command `seq-ref`.
  - Remove `seq-ref` as an alias for `sequence-ref`.
  - Duplicate `sequence-ref` tests for `seq-ref`.
  - Add test `sequence-ref-not-seq-ref` to make sure we didn't confuse the two
    anywhere.

- Make `sequence-index` compatible with `seq`s.

- Create custom sequence type for testing generic sequence commands.

- Change `loopy--obsolete-aliases` so that `seqf` and `seqingf` are obsolete
  aliases of `seq-ref`, not of `sequence-ref`.

- Update Org documentation, CHANGELOG, and README.
  - Note that not all generic sequences are mutable.
  - List `seqing-index` and `seqing-ref` as accepted names for Loopy Iter.

- Rename parsers `loopy--parse-seq-index` to `loopy--parse-sequence-index`,
  `loopy--parse-seq` to `loopy--parse-sequence`,
  and `loopy--parse-seq-ref` to `loopy--parse-sequence-ref`.

- Fix `sequence-ref-:by-index` (formerly `seq-ref-:by-index`) to that it
  correctly tests the generalized variable setter.

Closes issue #150.  This change might be improved if #136 is improved.
See also the PR #215.
okamsn added a commit that referenced this issue Jan 4, 2025
- Create command `seq`.
  - Remove `seq` as an alias for `sequence`.
  - Duplicate `sequence` tests for `seq`.
  - Add test `sequence-not-seq` to make sure we didn't confuse the two anywhere.

- Create command `seq-ref`.
  - Remove `seq-ref` as an alias for `sequence-ref`.
  - Duplicate `sequence-ref` tests for `seq-ref`.
  - Add test `sequence-ref-not-seq-ref` to make sure we didn't confuse the two
    anywhere.

- Make `sequence-index` compatible with `seq`s.

- Create custom sequence type for testing generic sequence commands.

- Change `loopy--obsolete-aliases` so that `seqf` and `seqingf` are obsolete
  aliases of `seq-ref`, not of `sequence-ref`.

- Update Org documentation, CHANGELOG, and README.
  - Note that not all generic sequences are mutable.
  - List `seqing-index` and `seqing-ref` as accepted names for Loopy Iter.

- Rename parsers `loopy--parse-seq-index` to `loopy--parse-sequence-index`,
  `loopy--parse-seq` to `loopy--parse-sequence`,
  and `loopy--parse-seq-ref` to `loopy--parse-sequence-ref`.

- Fix `sequence-ref-:by-index` (formerly `seq-ref-:by-index`) to that it
  correctly tests the generalized variable setter.

Closes issue #150.  This change might be improved if #136 is improved.
See also the PR #215.
okamsn added a commit that referenced this issue Jan 4, 2025
- Create command `seq`.
  - Remove `seq` as an alias for `sequence`.
  - Duplicate `sequence` tests for `seq`.
  - Add test `sequence-not-seq` to make sure we didn't confuse the two anywhere.

- Create command `seq-ref`.
  - Remove `seq-ref` as an alias for `sequence-ref`.
  - Duplicate `sequence-ref` tests for `seq-ref`.
  - Add test `sequence-ref-not-seq-ref` to make sure we didn't confuse the two
    anywhere.

- Make `sequence-index` compatible with `seq`s.

- Create custom sequence type for testing generic sequence commands.

- Change `loopy--obsolete-aliases` so that `seqf` and `seqingf` are obsolete
  aliases of `seq-ref`, not of `sequence-ref`.

- Update Org documentation, CHANGELOG, and README.
  - Note that not all generic sequences are mutable.
  - List `seqing-index` and `seqing-ref` as accepted names for Loopy Iter.

- Rename parsers `loopy--parse-seq-index` to `loopy--parse-sequence-index`,
  `loopy--parse-seq` to `loopy--parse-sequence`,
  and `loopy--parse-seq-ref` to `loopy--parse-sequence-ref`.

- Fix `sequence-ref-:by-index` (formerly `seq-ref-:by-index`) to that it
  correctly tests the generalized variable setter.

Closes issue #150.  This change might be improved if #136 is improved.
See also the PR #215.
okamsn added a commit that referenced this issue Jan 4, 2025
- Create command `seq`.
  - Remove `seq` as an alias for `sequence`.
  - Duplicate `sequence` tests for `seq`.
  - Add test `sequence-not-seq` to make sure we didn't confuse the two anywhere.

- Create command `seq-ref`.
  - Remove `seq-ref` as an alias for `sequence-ref`.
  - Duplicate `sequence-ref` tests for `seq-ref`.
  - Add test `sequence-ref-not-seq-ref` to make sure we didn't confuse the two
    anywhere.

- Make `sequence-index` compatible with `seq`s.

- Create custom sequence type for testing generic sequence commands.

- Change `loopy--obsolete-aliases` so that `seqf` and `seqingf` are obsolete
  aliases of `seq-ref`, not of `sequence-ref`.

- Update Org documentation, CHANGELOG, and README.
  - Note that not all generic sequences are mutable.
  - List `seqing-index` and `seqing-ref` as accepted names for Loopy Iter.

- Rename parsers `loopy--parse-seq-index` to `loopy--parse-sequence-index`,
  `loopy--parse-seq` to `loopy--parse-sequence`,
  and `loopy--parse-seq-ref` to `loopy--parse-sequence-ref`.

- Fix `sequence-ref-:by-index` (formerly `seq-ref-:by-index`) to that it
  correctly tests the generalized variable setter.

Closes issue #150.  This change might be improved if #136 is improved.
See also the PR #215.
okamsn added a commit that referenced this issue Jan 4, 2025
- Create command `seq`.
  - Remove `seq` as an alias for `sequence`.
  - Duplicate `sequence` tests for `seq`.
  - Add test `sequence-not-seq` to make sure we didn't confuse the two anywhere.

- Create command `seq-ref`.
  - Remove `seq-ref` as an alias for `sequence-ref`.
  - Duplicate `sequence-ref` tests for `seq-ref`.
  - Add test `sequence-ref-not-seq-ref` to make sure we didn't confuse the two
    anywhere.

- Make `sequence-index` compatible with `seq`s.

- Create custom sequence type for testing generic sequence commands.

- Change `loopy--obsolete-aliases` so that `seqf` and `seqingf` are obsolete
  aliases of `seq-ref`, not of `sequence-ref`.

- Update Org documentation, CHANGELOG, and README.
  - Note that not all generic sequences are mutable.
  - List `seqing-index` and `seqing-ref` as accepted names for Loopy Iter.

- Rename parsers `loopy--parse-seq-index` to `loopy--parse-sequence-index`,
  `loopy--parse-seq` to `loopy--parse-sequence`,
  and `loopy--parse-seq-ref` to `loopy--parse-sequence-ref`.

- Fix `sequence-ref-:by-index` (formerly `seq-ref-:by-index`) to that it
  correctly tests the generalized variable setter.

Closes issue #150.  This change might be improved if #136 is improved.
See also the PR #215.
okamsn added a commit that referenced this issue Jan 4, 2025
- Create command `seq`.
  - Remove `seq` as an alias for `sequence`.
  - Duplicate `sequence` tests for `seq`.
  - Add test `sequence-not-seq` to make sure we didn't confuse the two anywhere.

- Create command `seq-ref`.
  - Remove `seq-ref` as an alias for `sequence-ref`.
  - Duplicate `sequence-ref` tests for `seq-ref`.
  - Add test `sequence-ref-not-seq-ref` to make sure we didn't confuse the two
    anywhere.

- Make `sequence-index` compatible with `seq`s.

- Create custom sequence type for testing generic sequence commands.

- Change `loopy--obsolete-aliases` so that `seqf` and `seqingf` are obsolete
  aliases of `seq-ref`, not of `sequence-ref`.

- Update Org documentation, CHANGELOG, and README.
  - Note that not all generic sequences are mutable.
  - List `seqing-index` and `seqing-ref` as accepted names for Loopy Iter.

- Rename parsers `loopy--parse-seq-index` to `loopy--parse-sequence-index`,
  `loopy--parse-seq` to `loopy--parse-sequence`,
  and `loopy--parse-seq-ref` to `loopy--parse-sequence-ref`.

- Fix `sequence-ref-:by-index` (formerly `seq-ref-:by-index`) to that it
  correctly tests the generalized variable setter.

Closes issue #150.  This change might be improved if #136 is improved.
See also the PR #215.
okamsn added a commit that referenced this issue Jan 4, 2025
- Increase required version of Stream from 2.3.0 to 2.4.0.  This new version
  fixes some of the Seq.el features for streams and explicitly forbids
  modifying streams.

- Create command `seq`.
  - Remove `seq` as an alias for `sequence`.
  - Duplicate `sequence` tests for `seq`.
  - Add test `sequence-not-seq` to make sure we didn't confuse the two anywhere.

- Create command `seq-ref`.
  - Remove `seq-ref` as an alias for `sequence-ref`.
  - Duplicate `sequence-ref` tests for `seq-ref`.
  - Add test `sequence-ref-not-seq-ref` to make sure we didn't confuse the two
    anywhere.

- Make `sequence-index` compatible with `seq`s.

- Create custom sequence type for testing generic sequence commands.

- Change `loopy--obsolete-aliases` so that `seqf` and `seqingf` are obsolete
  aliases of `seq-ref`, not of `sequence-ref`.

- Update Org documentation, CHANGELOG, and README.
  - Note that not all generic sequences are mutable.
  - List `seqing-index` and `seqing-ref` as accepted names for Loopy Iter.

- Rename parsers `loopy--parse-seq-index` to `loopy--parse-sequence-index`,
  `loopy--parse-seq` to `loopy--parse-sequence`,
  and `loopy--parse-seq-ref` to `loopy--parse-sequence-ref`.

- Fix `sequence-ref-:by-index` (formerly `seq-ref-:by-index`) to that it
  correctly tests the generalized variable setter.

Closes issue #150.  This change might be improved if #136 is improved.
See also the PR #215.
okamsn added a commit that referenced this issue Jan 4, 2025
- Increase required version of Stream from 2.3.0 to 2.4.0.  This new version
  fixes some of the Seq.el features for streams and explicitly forbids
  modifying streams.

- Create command `seq`.
  - Remove `seq` as an alias for `sequence`.
  - Duplicate `sequence` tests for `seq`.
  - Add test `sequence-not-seq` to make sure we didn't confuse the two anywhere.

- Create command `seq-ref`.
  - Remove `seq-ref` as an alias for `sequence-ref`.
  - Duplicate `sequence-ref` tests for `seq-ref`.
  - Add test `sequence-ref-not-seq-ref` to make sure we didn't confuse the two
    anywhere.

- Make `sequence-index` compatible with `seq`s.

- Create custom sequence type for testing generic sequence commands.

- Change `loopy--obsolete-aliases` so that `seqf` and `seqingf` are obsolete
  aliases of `seq-ref`, not of `sequence-ref`.

- Update Org documentation, CHANGELOG, and README.
  - Note that not all generic sequences are mutable.
  - List `seqing-index` and `seqing-ref` as accepted names for Loopy Iter.

- Rename parsers `loopy--parse-seq-index` to `loopy--parse-sequence-index`,
  `loopy--parse-seq` to `loopy--parse-sequence`,
  and `loopy--parse-seq-ref` to `loopy--parse-sequence-ref`.

- Fix `sequence-ref-:by-index` (formerly `seq-ref-:by-index`) to that it
  correctly tests the generalized variable setter.

Closes issue #150.  This change might be improved if #136 is improved.
See also the PR #215.
okamsn added a commit that referenced this issue Jan 4, 2025
- Increase required version of Stream from 2.3.0 to 2.4.0.  This new version
  fixes some of the Seq.el features for streams and explicitly forbids
  modifying streams.

- Create command `seq`.
  - Remove `seq` as an alias for `sequence`.
  - Duplicate `sequence` tests for `seq`.
  - Add test `sequence-not-seq` to make sure we didn't confuse the two anywhere.

- Create command `seq-ref`.
  - Remove `seq-ref` as an alias for `sequence-ref`.
  - Duplicate `sequence-ref` tests for `seq-ref`.
  - Add test `sequence-ref-not-seq-ref` to make sure we didn't confuse the two
    anywhere.

- Make `sequence-index` compatible with `seq`s.

- Create custom sequence type for testing generic sequence commands.

- Change `loopy--obsolete-aliases` so that `seqf` and `seqingf` are obsolete
  aliases of `seq-ref`, not of `sequence-ref`.

- Update Org documentation, CHANGELOG, and README.
  - Note that not all generic sequences are mutable.
  - List `seqing-index` and `seqing-ref` as accepted names for Loopy Iter.

- Rename parsers `loopy--parse-seq-index` to `loopy--parse-sequence-index`,
  `loopy--parse-seq` to `loopy--parse-sequence`,
  and `loopy--parse-seq-ref` to `loopy--parse-sequence-ref`.

- Fix `sequence-ref-:by-index` (formerly `seq-ref-:by-index`) to that it
  correctly tests the generalized variable setter.

Closes issue #150.  This change might be improved if #136 is improved.
See also the PR #215.
github-actions bot added a commit that referenced this issue Jan 4, 2025
#215)

This commit was copied from the master branch.

Commit: 95f7d12
Author: okamsn <[email protected]>
Date: 2025-01-04 23:03:14 +0000

Create loop commands for generic (`seq.el`) sequences. (#215)

- Increase required version of Stream from 2.3.0 to 2.4.0.  This new version
  fixes some of the Seq.el features for streams and explicitly forbids
  modifying streams.

- Create command `seq`.
  - Remove `seq` as an alias for `sequence`.
  - Duplicate `sequence` tests for `seq`.
  - Add test `sequence-not-seq` to make sure we didn't confuse the two anywhere.

- Create command `seq-ref`.
  - Remove `seq-ref` as an alias for `sequence-ref`.
  - Duplicate `sequence-ref` tests for `seq-ref`.
  - Add test `sequence-ref-not-seq-ref` to make sure we didn't confuse the two
    anywhere.

- Make `sequence-index` compatible with `seq`s.

- Create custom sequence type for testing generic sequence commands.

- Change `loopy--obsolete-aliases` so that `seqf` and `seqingf` are obsolete
  aliases of `seq-ref`, not of `sequence-ref`.

- Update Org documentation, CHANGELOG, and README.
  - Note that not all generic sequences are mutable.
  - List `seqing-index` and `seqing-ref` as accepted names for Loopy Iter.

- Rename parsers `loopy--parse-seq-index` to `loopy--parse-sequence-index`,
  `loopy--parse-seq` to `loopy--parse-sequence`,
  and `loopy--parse-seq-ref` to `loopy--parse-sequence-ref`.

- Fix `sequence-ref-:by-index` (formerly `seq-ref-:by-index`) to that it
  correctly tests the generalized variable setter.

Closes issue #150.  This change might be improved if #136 is improved.
See also the PR #215.
github-actions bot added a commit that referenced this issue Jan 4, 2025
#215)

This commit was copied from the master branch.

Commit: 95f7d12
Author: okamsn <[email protected]>
Date: 2025-01-04 23:03:14 +0000

Create loop commands for generic (`seq.el`) sequences. (#215)

- Increase required version of Stream from 2.3.0 to 2.4.0.  This new version
  fixes some of the Seq.el features for streams and explicitly forbids
  modifying streams.

- Create command `seq`.
  - Remove `seq` as an alias for `sequence`.
  - Duplicate `sequence` tests for `seq`.
  - Add test `sequence-not-seq` to make sure we didn't confuse the two anywhere.

- Create command `seq-ref`.
  - Remove `seq-ref` as an alias for `sequence-ref`.
  - Duplicate `sequence-ref` tests for `seq-ref`.
  - Add test `sequence-ref-not-seq-ref` to make sure we didn't confuse the two
    anywhere.

- Make `sequence-index` compatible with `seq`s.

- Create custom sequence type for testing generic sequence commands.

- Change `loopy--obsolete-aliases` so that `seqf` and `seqingf` are obsolete
  aliases of `seq-ref`, not of `sequence-ref`.

- Update Org documentation, CHANGELOG, and README.
  - Note that not all generic sequences are mutable.
  - List `seqing-index` and `seqing-ref` as accepted names for Loopy Iter.

- Rename parsers `loopy--parse-seq-index` to `loopy--parse-sequence-index`,
  `loopy--parse-seq` to `loopy--parse-sequence`,
  and `loopy--parse-seq-ref` to `loopy--parse-sequence-ref`.

- Fix `sequence-ref-:by-index` (formerly `seq-ref-:by-index`) to that it
  correctly tests the generalized variable setter.

Closes issue #150.  This change might be improved if #136 is improved.
See also the PR #215.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request upstream Applies to a dependecy
Projects
None yet
Development

No branches or pull requests

1 participant