Incidencia #40944

player-status-resist のforループ変数スコープを狭める

Abrir Fecha: 2020-11-10 19:08 Última actualización: 2021-10-29 19:43

Informador:
Propietario:
Estado:
Cerrado
Componente:
(Ninguno)
Prioridad:
2
Gravedad:
2
Resolución:
Fixed
Fichero:
Ninguno

Details

#40514 の関連で、forループの外側で「int i;」が宣言されています

しかしループの外側でiの値を使っていないので、forループ内で宣言するように修正願います

※ ファイル中に4箇所同様の宣言があります

Ticket History (3/6 Histories)

2020-11-10 19:08 Updated by: hourier
  • New Ticket "player-status-resist のforループ変数スコープを狭める" created
2020-11-11 08:18 Updated by: deskull
  • Resolución Update from Ninguno to Accepted
2020-11-11 08:22 Updated by: deskull
Comentario

今更検索しましたが、japanese.cなど少数の例外を除いてほぼ大多数はそういう方針になっていますね。 コーディング規約、とまではいかんでもこれを推奨の慣例としておくつもりでやっておきます。

2020-11-25 08:15 Updated by: deskull
Comentario

話が遅れたがjapanese.cなども含めて自分の手を付けてない部分も今日中位に直しておく。

2020-12-08 08:24 Updated by: deskull
Comentario

今日中どころではないが見直し中。随所に問題の箇所は多数見られるが、中には複雑な分岐をforで絡めているところもあり迂闊に直すとエンバグになる可能性は高い。 検索の上方から順にいくつか手を加えてのを除き後バージョンに回し優先度も下げる。

2021-10-29 19:43 Updated by: hourier
  • Estado Update from Open to Cerrado
  • Resolución Update from Accepted to Fixed
Comentario

最終コメントから10ヶ月が経過し、既に相当量のリファクタリングがなされている
他ファイルも含めて十分に無実化したと思われるのでクローズ

Attachment File List

No attachments

Editar

Please login to add comment to this ticket » Entrar