typelevel/spire

Use of bug in `cfor` macro

Open

#1,225 opened on 2022年11月25日

GitHub で見る
 (7 comments) (0 reactions) (0 assignees)Scala (1,775 stars) (238 forks)batch import
bughelp wanted

説明

The cfor macro accidentally generated the desired code by using a bug in beta-reduction. This bug will be patched https://github.com/lampepfl/dotty/pull/16390 and cfor will need to be updated.

See https://github.com/dotty-staging/spire/compare/7f630c0209e327bdc782ade2210d8e4b916fddcc...99ea909d086e28e85dbdf8aa78ef0a83bf873405#diff-9cd0f3a9df06a2589472aa8dc960fd2a65a2b65f2f8c257addda1c86ebc31469

コントリビューターガイド

技術スタック
scala
領域
tooling
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Scalamacros
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
25
調査方針
Check the linked dotty PR (https://github.com/lampepfl/dotty/pull/16390) for the beta reduction fix. Then update the `cfor` macro in Spire to use the proper behavior without relying on the bug. Examine the diff in the linked fork (https://github.com/dotty staging/spire/compare/...) for reference.