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

Signal a warning for incompatible accumulation initializations. #203

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

okamsn
Copy link
Owner

@okamsn okamsn commented Aug 20, 2024

  • Rename error loopy-incompatible-accumulations to
    loopy-incompatible-accumulation-types.

  • Signal a warning a when there are multiple expected starting values for an
    accumulation variable. In the future, this will signal an error.

    For example, sum and multiply are compatible in that they use numbers,
    but the former expects 0 and the latter expects 1. The warning
    and future error can be avoided by using the with special macro argument.

okamsn added a commit that referenced this pull request Aug 20, 2024
- Rename error `loopy-incompatible-accumulations` to
  `loopy-incompatible-accumulation-types`.

- Create the error `loopy-incompatible-accumulation-initializations`,
  but don't use it yet.

- Signal a warning an there are multiple expected starting values for an
  accumulation variable.  In the future, this will signal an error.

See also this PR #203 and issue #169.
okamsn added a commit that referenced this pull request Aug 20, 2024
- Rename error `loopy-incompatible-accumulations` to
  `loopy-incompatible-accumulation-types`.

- Create the error `loopy-incompatible-accumulation-initializations`,
  but don't use it yet.

- Signal a warning an there are multiple expected starting values for an
  accumulation variable.  In the future, this will signal an error.

See also this PR #203 and issue #169.
- Rename error `loopy-incompatible-accumulations` to
  `loopy-incompatible-accumulation-types`.

- Create the error `loopy-incompatible-accumulation-initializations`,
  but don't use it yet.

- Signal a warning an there are multiple expected starting values for an
  accumulation variable.  In the future, this will signal an error.

- Update Org documentation and Texinfo file to note this new warning.

See also this PR #203 and issue #169.
@okamsn okamsn merged commit b088504 into master Aug 20, 2024
4 checks passed
github-actions bot added a commit that referenced this pull request Aug 20, 2024
…lizations. (#203)

This commit was copied from the master branch.

Commit: b088504
Author: okamsn <[email protected]>
Date: 2024-08-20 01:41:17 +0000

Signal a warning for incompatible accumulation initializations. (#203)

- Rename error `loopy-incompatible-accumulations` to
  `loopy-incompatible-accumulation-types`.

- Create the error `loopy-incompatible-accumulation-initializations`,
  but don't use it yet.

- Signal a warning an there are multiple expected starting values for an
  accumulation variable.  In the future, this will signal an error.

- Update Org documentation and Texinfo file to note this new warning.

See also this PR #203 and issue #169.
github-actions bot added a commit that referenced this pull request Aug 20, 2024
…lizations. (#203)

This commit was copied from the master branch.

Commit: b088504
Author: okamsn <[email protected]>
Date: 2024-08-20 01:41:17 +0000

Signal a warning for incompatible accumulation initializations. (#203)

- Rename error `loopy-incompatible-accumulations` to
  `loopy-incompatible-accumulation-types`.

- Create the error `loopy-incompatible-accumulation-initializations`,
  but don't use it yet.

- Signal a warning an there are multiple expected starting values for an
  accumulation variable.  In the future, this will signal an error.

- Update Org documentation and Texinfo file to note this new warning.

See also this PR #203 and issue #169.
@okamsn okamsn deleted the ambig-init branch September 14, 2024 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant