eigenlsp: definition/symbol range for a function covers 'define', not the name
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 72/100
調査の方向性
Start in src/eigenlsp.c around lines 1209, 1261, 1339, and 1725, tracing how a symbol's col is used for definition, documentSymbol, workspaceSymbol, reference, and rename ranges. Add or update coverage in tests/test_lsp.py for definition, documentSymbol, and references on a define, then run that test and confirm ranges use the helper name rather than define.
索引モデルが issue の本文から書いたものです。
説明
Found by the builder on #1368.
textDocument/definition on a call to helper returns {start:{0,0}, end:{0,6}} for define helper(a) as:. That range covers the define keyword. s->col is the define STATEMENT's column, and the end is computed as col + strlen(name). This happens with space indentation too. The same s->col feeds documentSymbol, workspaceSymbol and the symbol-sourced reference and rename ranges (src/eigenlsp.c around lines 1209, 1261, 1339, 1725). Any edit built from those ranges would rewrite define.
Done when
- definition of
helperreturns{0,7}..{0,13}(the name's bytes). A test in tests/test_lsp.py covers definition, documentSymbol and references on adefine. - Every range built from a symbol's
colis audited, and each one either uses the name's column or states why the statement column is right.
- 主要言語
- C
- スター
- 3
- フォーク
- 7
- 平均マージ
- 8時間 43分
- マージ済み PR（30日）
- 253
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
InauguralSystems/EigenScript のほかの issue
-
area:gates bug good first issue
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
InauguralSystems/EigenScript#1782 ·
メンテナーはふだん 1 日以内に返信
-
eigenscript.h declares 8 observer functions that no longer exist対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンarea:observer enhancement found-by:cold-read good first issue
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
InauguralSystems/EigenScript#1781 ·
メンテナーはふだん 1 日以内に返信
-
Amalgamation compiles with warnings at -O0 (-Wreturn-type) and -O1 (-Wmaybe-uninitialized)対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンarea:embed bug found-by:code-review
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
InauguralSystems/EigenScript#1775 ·
メンテナーはふだん 1 日以内に返信
-
UB: (int) cast of a user double outside int range in dispatch / vm_index_is_int / flat_index_atオープンarea:runtime-vm
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
InauguralSystems/EigenScript#1748 ·
メンテナーはふだん 1 日以内に返信
-
scan_int_tokens line/col fields are untested対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンarea:stdlib found-by:critic kind:gate-defect
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
InauguralSystems/EigenScript#1693 ·
メンテナーはふだん 1 日以内に返信
InauguralSystems/EigenScript の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
libsdl-org/SDL#16530 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
FriendsofECCE/ECCE#336 · コメント 1 件 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
fw876/helloworld#2067 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
immortalwrt/immortalwrt#2501 ·