Ticket #39962

files.cの分割

Eröffnet am: 2020-01-29 23:00 Letztes Update: 2021-01-02 09:58

Auswertung:
Verantwortlicher:
Status:
Geschlossen
Komponente:
(Keine)
Priorität:
3
Schweregrad:
3
Lösung:
Accepted
Datei:
Keine

Details

6000行以上ある上に、ファイルI/Oとは直接関係のない関数が大量に押し込まれている

意味のある単位に分割するところから始める

・画面表示を担当している関数 (display_*)

・シグナルハンドラを担当している関数 (signals_*)

更に、値の更新と画面表示が同一の関数に混じっているのでコールバックで依存の方向を片方向に制限する

dump_aux_display_player() やshow_info() の中にあるdisplay_player() 等

Ticket-Verlauf (3/5 Historien)

2020-01-29 23:00 Aktualisiert von: hourier
  • New Ticket "files.cの分割" created
2020-01-29 23:04 Aktualisiert von: hourier
Kommentar

#39912 の関係と思われるが、関数分離を試みたところコンパイルエラーでまともに動作しなかった

現時点では一旦放置して、deskull 氏に呼び出し関係の調査を先に行って頂くこととする

当面はコールバック関数を呼び出せるような構造の設計を考えてみる

2020-05-19 23:48 Aktualisiert von: hourier
  • Status Update from Offen to Geschlossen
Kommentar

少し強引だがfiles-util.c/h というファイル名にし、その他の関数は分割しきった

今後は個別のファイルについて必要があれば追加でリファクタリングを行うものとする

2021-01-02 09:56 Aktualisiert von: deskull
2021-01-02 09:58 Aktualisiert von: deskull
  • Lösung Update from Gefixt to Accepted

Dateianhangliste

Keine Anhänge

Bearbeiten

Please login to add comment to this ticket » Anmelden