spells1.cの分割

• Reporter: hourier
• Owner: hourier
• Type: リファクタリング
• Status: Closed
• Component: (None)
• MileStone: 変愚蛮怒3.0.0alphaリリース (closed)
• Priority: 3
• Severity: 3
• Resolution: Accepted
• File: None

Details

files.c と同様

6000行以上あるので何とかしたい

しかしながらインクルード関係のエラーが出ている模様なので、先に#39912 を片付けて頂く方針とする

こちらはスペルの種別ごとに関数分割を試みる方針とする、分割先からspells1.cを呼ばないようにコールバックの用意も必要

(まずはコメントの整理から始めようと思う……)

Ticket History (3/4 Histories)

2020-01-29 23:18 Updated by: hourier

• New Ticket "spells1.cの分割" created

2020-05-09 16:16 Updated by: hourier

• Status Update from Open to Closed

Comment

引数をそのまま参照したり書き換えたりしていて対処しにくいproject() をprocess-effect.c/h に隔離して、他は順次解体した

process-effect.c/h もやや重量級のファイルだが、これ以上下手にリファクタリングすると見つけにくい不具合の原因になりそうだったので一旦取りやめ

今後は個別のファイルについてリファクタリングを行うものとし、チケットを完了させる

2021-01-02 09:57 Updated by: deskull

• Milestone Update from 変愚蛮怒3.0.0リファクタリング (deleted) to 変愚蛮怒3.0.0alphaリリース (closed)
• Resolution Update from None to Fixed

2021-01-02 09:58 Updated by: deskull

• Resolution Update from Fixed to Accepted