Ticket #39670

externs.hに書かれている出典元ファイルが誤っている

Open Date: 2019-10-11 12:59 Last Update: 2021-01-01 23:40

Reporter:
Owner:
Type:
Status:
Closed
Component:
(None)
Priority:
5 - Medium
Severity:
5 - Medium
Resolution:
Fixed
File:
None

Details

チケット #39669 の調査を行う際に、externs.h を参照したが、以下にあげる関数について出典元ファイルが誤っていた。 おそらく他にもあると思われるが、全部は調査していない。

xtra1.cとあるがview-mainwindow.cである関数

update_playtime
handle_stuff
update_output

xtra2.cとあるがview-mainwindow.cにある関数

get_screen_size

spells1.cとあるがspells-diceroll.cにある関数

beam_chance

Ticket History (3/9 Histories)

2019-10-11 12:59 Updated by: osakanataro
  • New Ticket "externs.hに書かれている出典元ファイルが誤っている" created
2019-10-11 22:23 Updated by: deskull
Comment

申し訳ない、この辺もあることは織り込んで修正中です。全部解決するまではしばらくお待ちください。

2020-01-07 01:02 Updated by: deskull
Comment

上記に書かれている齟齬までは https://osdn.net/projects/hengband/scm/git/hengband/commits/e94c29bf0d85feb3a425caf053831240a2be0ea3 で修正しました。他もないか把握するまではチケット残します。

2020-01-25 19:06 Updated by: hourier
Comment

少し前のコミットで恐縮ですが、object.h にあったいくつかのメソッドをplayer-inventory.hへ移しました

私が把握している限りでは、inet.h にある下記は全てchuukei.c の関数とグローバル変数です

・prepare_browse_movie_aux()

・prepare_browse_movie()

・browse_movie()

・browsing_movie (bool型)

但しこれをchuukei.hへ移すとh-type.hのインクルードに問題があるのかconcptr型を正しく認識しなくなる事象が起きています

お手数ですがご確認下さい

そしてexterns.h を削除した関係上、このチケットは閉じて個別のヘッダファイルについてチケット化していくのが良いと思います

2020-01-25 19:09 Updated by: hourier
Comment

雑多な修正も混じっていますが、下記で修正しました↓

https://osdn.net/projects/hengband/scm/git/hengband/commits/0cf2dd3eb6f1dea20ea648a0bd236f4294666db1

2020-02-23 03:45 Updated by: deskull
  • Owner Update from deskull to hourier
2020-02-23 12:01 Updated by: hourier
  • Resolution Update from None to Fixed
  • Owner Update from hourier to deskull
Comment

以前のやり方が悪かったのか、#39912 でインクルードが整理されたからなのか、何はともあれ無事コンパイルが通るようになったので修正作業を実施しました

マージ確認後、クローズ願います

2020-04-25 00:09 Updated by: deskull
  • Status Update from Open to Closed
Comment

言及遅れましたが、マージ済ませました。ありがとうございました。

Attachment File List

No attachments

Edit

Please login to add comment to this ticket » Login