New Private field method access issue [Workaround]
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 30/100
調査の方向性
NewFieldsRewriterから始め、Unityのホットリロードワークフローで報告されたprivateフィールドへのアクセスケースを再現します。_instanceNewFieldへの直接アクセスとローカル変数を使った回避策を比較し、そのうえで、Gizmos.DrawWireSphere(_lastCamState.FinalPosition, BodyRadius)のような使用箇所が、構文のクリーンアップを必要とせず一貫して書き換えられることを確認します。
索引モデルが issue の本文から書いたものです。
説明
Current Workaround
So it seems this is the workaround for now. Any direct access the new field breaks the rewritter.
_instanceNewField.DoesWhatYouWant(); //<- fails to rewrite
var myLocalNewField = _instanceNewField; // <- this gets correctly rewritten
myLocalNewField.DoesWhatYouWant(); //<- All good
Then on your next recompilation round you can clean up the syntax.
Issue
I'm not sure if it's a regression and what would be its reason, but most of the time nowadays adding private fields fails as they are not rewritten correctly in every places. Example bellow.
Added
private CameraState _lastCamState;
One of the correctly rewritten part I believe
FastScriptReload.Scripts.Runtime.TemporaryNewFieldValues.ResolvePatchedObject<Indus.Client.Cameras.CinemachineBodyCollider__Patched_>(this)._lastCamState = state;
Some other places not rewritten
Gizmos.DrawWireSphere(_lastCamState.FinalPosition, BodyRadius);
It is the NewFieldsRewriter job to perform the rewrite now not sure why it skip some cases.
- 主要言語
- C#
- スター
- 2.2k
- フォーク
- 167
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトの環境構築ファイルはまだ確認していません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
handzlikchris/FastScriptReload のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
handzlikchris/FastScriptReload#202 · リアクション 2 件 ·
-
csc.rsp supportオープン
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
-
難易度 4/5 3〜5日 初心者へのやさしさ 38/100
handzlikchris/FastScriptReload#203 · コメント 4 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 42/100
handzlikchris/FastScriptReload#197 · コメント 1 件 ·
handzlikchris/FastScriptReload の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
microsoft/onnxruntime-genai#2633 ·
メンテナーはふだん 1 日以内に返信
-
python triage
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
microsoft/semantic-kernel#14491 ·
メンテナーはふだん 2 日以内に返信
-
area:jobads-cv FE mvp P3
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
klasolsson81/jobbliggaren#1878 ·
メンテナーはふだん 1 日以内に返信
-
bug help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
SubtitleEdit/subtitleedit#15331 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信