Merge pull request #138 from Hourier/release/3.0.0Alpha6

Release/3.0.0 alpha6

[Version] 3.0.0Alpha6

Merge pull request #113 from taotao54321/fix-insufficient-mbe-info

[Fix] mbe_info の配列外参照を修正

Merge pull request #137 from dis-/feature/Fix-check_floor_item_tag_inventory

[Fix] アイテム選択時にクラッシュ #117

Merge pull request #136 from dis-/feature/Refactor-Monster-Lore-Mode

[Refactor] モンスターの思い出表示のモードをenumに変更

Merge pull request #123 from backwardsEric/autopick-junk-key

English autopick adjustments for "junks" to "junk" change

[Fix] アイテム選択時にクラッシュ #117

コマンド繰り返しでタグ付アイテムを使用する行動を繰り返すとき、個数が0になっても該当アイテムを選択し続けていた。
check_floor_item_tag_inventory()の処理がv2.2.1と異なっていたため、従来の挙動通りになるように再度実装した。
具体的には、コマンド繰り返しでタグ付アイテムの取得に失敗したときに以降の処理を打ち切ってFALSEを返すように変更した。

[Refactor] モンスターの思い出表示のモードをenumに変更

可読性が厳しかったため、enumにして条件を整理した。
通常用、調査時専用、デバッグ専用の3種類のモードがある。

Merge pull request #132 from Hourier/feature/Refactor-Monster-Alpha6

Feature/refactor monster alpha6

[Implement] #41345 バンシーに悲鳴フラグを追加した / Added SHRIEK flag to Banshee

[Implement] #41042 バーゲストのレアリティを1から2に下げた / Decreased the rarity of barghest from 1 to 2

[WIP] [Refactor] #41042 赤チォコヴォのパラメータを調整した / Adjusted the parameters of red chiokovo

[Implement] #41042 投稿頂いたモンスター群の修正依頼を実装した/ Adjusted the parameters of monsters by poster

[Refactor] #41042 相談により『ソラリス』のパラメータを変更した / Adjusted the parameters of Solaris by meeting

[Refactor] #41042 グレーター・ファルメル・スクルカーからPOWERFULフラグを削除した / Removed POWERFUL flag from Greater falmer skulker

[Refactor] #41042 lainのパラメータを調整した / Adjusted the parameters of lain

[Refactor] #41042 『アルドゥイン』のパラメータを調整した / Adjusted the parameters of Alduin

[Fix] #41042 二周目の泡に関するモンスターの表記ゆれを修正した / Resolved the issue that monster notation fluctuations related to aqua illusions in 2nd lap

Merge pull request #122 from backwardsEric/english-wizard-edits

3.0.0.0: minor English edits for src/wizard

Merge pull request #121 from backwardsEric/english-view-edits

3.0.0.0: minor English edits for src/view

To match the "junks" to "junk" change for the autopick key in the source code, change the default English autopick preference file and the autopick help file.

Fix English spoiler label for TR_AGGRAVATE.

Fix English typo.

Replaced "Bare hand" with "Barehanded".

For the English self information about demon lords, put "nether" after "breathe" to be more idiomatic and consistent with the breath messages elsewhere.

Merge pull request #103 from dis-/feature/Fix-Update-Message-Window

[Fix] 床置きアイテム自動破壊時にメッセージウインドウが遅れる #92

Merge pull request #116 from dis-/feature/Fix-Blue-Mage-Firebolt

[Fix] 青魔導士のファイア・ボルトがMP消費ターン消費なし #115

[Fix] 青魔導士のファイア・ボルトがMP消費ターン消費なし #115

ファイア・ボルトのみコスト消費判定が反転している。
該当箇所を修正。

Merge pull request #114 from taotao54321/fix-music-after-home-and-museum

[Fix] 我が家/博物館から出た際に辺境の地のBGMが鳴るバグを修正

Merge pull request #102 from dis-/feature/Fix-Load-Save

[Fix] セーブファイルを指定して起動した時にpath指定がおかしくて起動しない #101

[Fix] 我が家/博物館から出た際に辺境の地のBGMが鳴るバグを修正

我が家/博物館の場合、一時的に現在地 (player_ptr->town_num) を辺境の地に
偽装して処理するが、その偽装を解除する前に select_floor_music() していた
のが原因。

そもそも player_ptr->town_num 自体を書き換える必要があるのかは疑わしい。
この偽装がなければ inner_town_num グローバル変数も消せるはず。
今後のリファクタ課題。

[Fix] mbe_info の要素が足りていなかったのを修正

RBE_INERTIA, RBE_STUN, RBE_FLAVOR に対応する要素を追加。
RBE_FLAVOR はただのフレーバーなので、(0, GF_NONE) を設定した。

INERTIA(減速)、STUN(朦朧)について:
今のところこれらの効果で爆発するモンスターはいないので、explode_type は
他の効果に倣って GF_MISSILE を設定した。
power は命中率に影響する。とりあえず他のデバフ効果の power が 0~10 程度
なので 5 としておいた。

[Refactor] rbm_type, rbe_type に enum バリアント数を追加

NB_<型名> というバリアントを enum のバリアント数を表す定数として定義する。
そして、配列サイズなどバリアント数が必要な箇所ではこの定数を使う。

従来は配列サイズなどを別個に定数定義していたため、enum のバリアントが増
えたのに配列サイズの方を変え忘れて配列外参照を起こしたりしていた。
本修正により自動で配列サイズの方も変わるので、少なくとも配列外参照はなく
なる。ただし、配列内容の追加忘れまでは防げない(初期化時に要素が足りなく
ても単に0初期化されるため。今後の課題)。

Merge pull request #110 from taotao54321/fix-unexpected-rbe-none

[Fix] 打撃効果 RBE_NONE の警告メッセージが出るバグを修正

Merge pull request #109 from taotao54321/fix-typo-rumors-j

[Fix] rumors_j.txt の typo を修正。クイスルグス -> クイルスルグ

[Refactor] 実質 bool 型の関数の戻り型が int だったのを修正

[Fix] 打撃効果 RBE_NONE の警告メッセージが出るバグを修正

r_info.txt の "B:" トークンで method だけが定義されていて effect がない
ものがあり、その場合 effect が RBE_NONE になっていた。
これに該当するのは「金をせがむ」「よだれをたらす」などのフレーバー的行動
のみなので、Discord での議論も踏まえて RBE_FLAVOR を新設し、RBE_NONE は
従来通り異常値扱いにした。

とりあえず v2.2.1 の仕様を踏襲し、RBE_FLAVOR は必中扱いとする。
ただしこれだとフレーバー的行動でも対邪悪結界や変わり身が発動するので、そ
の辺は議論の余地ありかも。
また、rbe_type のバリアントが増えたので MAX_MBE, mbe_info も更新する必要
があるが、それは #105 に回す。

Merge pull request #95 from taotao54321/fix-monap-type-ap-cnt

[Refactor] mon_type、monap_type の ap_cnt メンバを除去

[Fix] rumors_j.txt の typo を修正。クイスルグス -> クイルスルグ

Merge pull request #104 from habu1010/feature/Fix-pkg-config-error

[fix] pkg-config のチェックを追加 #12

Merge pull request #76 from taotao54321/fix-left-shift-by-31

[Fix] s64b_RSHIFT() の未定義動作を修正

[Fix] 床置きアイテム自動破壊時にメッセージウインドウが遅れる #92

マクロ動作完了後、キー入力前のタイミングで必要項目を再描画するようにしました。
全サブウィンドウに対してこの処理を実行するので、視界内のモンスター表示のみを強制描写していた処理が不要になりました。
fix_monster_list()の引数を減らし、never_freshフラグを操作して描写処理を行います。

[fix] pkg-config のチェックを追加 #12

./configureに --enable-xft と --disable-worldscore の両方の
スイッチを渡すと、XFTの検出ができずエラーとなる。
現象から判断するに、先にlibcurlの検出をしていると
XFTの検出もできる模様。
他のソフトウェアのconfigure.acの例を見て、
PKG_PROG_PKG_CONFIG (pkg-configの検出)を追加すると
--enable-xft と --disable-worldscore の両方を渡しても
正常に検出できるようになった。
正直 configure.ac は複雑すぎてわからないところだらけで
PKG_PROG_PKG_CONFIG の追加でOKになった理由も
よくわからないが、どのみち pkg-config の検出は
あったほうが良いと思われるので追加して修正とする。

increase_blow_type_seen() に doxygen コメントを追加。

Merge pull request #97 from habu1010/feature/Fix-flicker-monster-list

[fix] モンスター一覧を表示すると画面がちらつく#96

[Fix] 64bitシフト演算を関数化(未定義動作の回避、仕様の明確化)

従来の64bitシフト関数には以下のような問題があった:

* s64b_RSHIFT() は、(V1<<(32-(N))) の部分で未定義動作が起こりうる。
  例えば V1 != 0 && N == 1 のとき、V1 が 31 回左シフトされてオーバーフローする。
  現状では敵を殴るだけでこれが発生するため、ubsan を用いたデバッグがしにくい。
* シフト量 0 を渡すと 32 回のシフトが発生して未定義動作となる。
* マクロで実装されているため型チェックが甘く、assert なども入れにくい。
* 引数の範囲についての仕様が書かれていない。

非マクロ関数を設けてこれらを解決する。
シフト対象は64bit非負整数、シフト量は [0,31] という仕様にする。
既存コードのシフト関数呼び出しを見る限り、これで要求は満たしているはず。

[Fix] セーブファイルを指定して起動した時にpath指定がおかしくて起動しない #101

コマンドライン等からセーブファイルを起動するときcheck_for_save_file()にて引数を判断している。
ここでグローバル変数savefileが空であることを前提にしてstrcatしていたためパスを正常に指定できなくなっていた。
コマンドライン等からの起動であるとき、INIファイルを無視して指定セーブを開く挙動が正しい。
よってsavefileを上書きするstrcpyを使用するよう変更した。

ついでに、初期化中ダイアログを初期化処理未完了の場合のみ表示するよう変更。

Merge pull request #100 from backwardsEric/english-term-edits

3.0.0.0: minor English edits for src/term

Merge pull request #99 from backwardsEric/english-store-edits

3.0.0.0: minor English edits for src/store

Merge pull request #84 from backwardsEric/english-spell-realm-edits

3.0.0.0: minor English edits for src/spell-realm

For the English symbol descriptions of shop entrances, add "the" to be idiomatic (except for the player's home) and match the name (both words and capitalization) to what's in f_info.txt.

To be more idiomatic, change "don't have any item" to "don't have any items".

Fill out English message for rejecting purchases in the museum.

To be more idiomatic, preface "Museum" with "the".

[fix] モンスター一覧を表示すると画面がちらつく#96

Linux/UNIX環境のGCU版で、サブウィンドウに視界内のモンスター
一覧を表示していると、毎ターン画面がちらつく。
原因は、視界内のモンスター一覧で最初に画面を全消去するため
term_clear()を呼んでいる事にある。
GCU版では、term_clearにより画面の全消去が入ると、端末全体の
再描画が行われるようになっているので、モンスター一覧のみで
なく画面全体の再描画が行われてしまう。

対応策として、term_clear()による全消去は行わずモンスター一覧を
表示する時に1行ずつterm_eraseで消去してから書くようにする。
これはサブウィンドウの持ち物一覧表示などでも使用している
方法で、先に全消去するよりもパフォーマンスもいいのではないかと
推察する。

[Refactor] mon_type、monap_type の ap_cnt メンバを除去

Fixes #63.

Merge pull request #90 from habu1010/feature/Expand-screen-dump-size

[update] 大きなサイズのマップもスコアサーバへ送信できるようにしたい #43

Merge pull request #35 from taotao54321/fix-chameleon-cave-monster-generation

[Fix] カメレオン洞に通常モンスターが生成されるのを修正

Merge pull request #94 from taotao54321/fix-rumors-j-missing-newline

[Fix] rumors_j.txt 内の改行忘れを修正

Merge pull request #36 from taotao54321/fix-wrong-evasion-message

[Fix] 打撃回避判定のバグ修正

[Fix] rumors_j.txt 内の改行忘れを修正

Fixes #93.

[Fix] 対邪悪結界で打撃を撃退した際は落馬/変わり身処理を行わない

process_monster_attack_hit() の戻り値を無視していたため、撃退したのに落
馬/変わり身処理が発動するようになっていた。
なお、#21 での議論により、撃退時も思い出処理は行うこととした (v2.2.1 か
らの仕様変更)。

[update] 大きなサイズのマップもスコアサーバへ送信できるようにしたい #43

生成されるスクリーンダンプのサイズ制限を、256KiB→1MiBに拡張する。
また、セーブファイルからスクリーンダンプをロードする時に
スタック領域に1MiBのバッファを積むことはできないので、
ヒープ領域にバッファを用意する。

Merge pull request #87 from habu1010/feature/Fix-window-width-overflow

[fix] 横幅をめちゃくちゃ広くすると、インベントリの表示などが乱れる #86

[fix] 横幅をめちゃくちゃ広くすると、インベントリの表示などが乱れる #86

ターミナルのサイズをbyteからTERM_LEN(int)に拡張したのに、
座標を代入する時に(byte)に不要なキャストをしているのが原因。
キャストせずそのまま代入するようにする。

Merge pull request #85 from backwardsEric/english-status-edits

3.0.0.0: minor English edits for src/status

Merge pull request #70 from habu1010/feature/Fix-object-no-choice

[fix] #65 未初期化の変数を参照している

For "x resistant" insert a "-" (combining noun and adjective to from an adjective).  Alternatively could use "resistant to x", but that's longer.

Modify test for "a" or "an" in race change message to allow for the possibility that effect_msg could start with a vowel (current callers only pass messages that start with consontants).

To avoid the impression that there are three regeneration states (fast, normal, slow), reword the English message displayed when fast regeneration lapses.

Merge pull request #80 from taotao54321/fix-ubsan-spell-is-inate

[Fix] spell_is_inate() の未定義動作を修正

Merge pull request #78 from taotao54321/fix-x11-convert-to-euc

[Fix] X11 ドライバで日本語入力時にゴミが入ることがあるのを修正

Merge pull request #74 from taotao54321/fix-x11-valgrind

[Fix] X11 ドライバで valgrind 警告が出なくなるようにした

For flavor, specialize the stopped bleeding message in English for androids, like the Japanese version does.

For flavor, change the English messages about blinding for androids to be closer to what Google Translate gives for the Japanese messages.

Drop "the" in front of "Judgement" to match what appears to be the normal way of way referring to the Tarot card.

"Resume" seems like a better fit than "restart" for the message about singing again.

Move negaative from object to verb in English message for no hex spell to stop.

Reword English message for the lapse of "An Eye for an Eye".

Fix the English messages to select an item with the polish shield spell.

Merge pull request #81 from backwardsEric/english-spell-kind-edits

3.0.0.0: minor English edits for src/spell-kind

Got rid of extranous capitalization in the English immunity onset message.  Dropped the extraneous "do" in that message for the case of an unhandled element type.

Change "light item" to light" for message about the arcane spell to refuel a lantern or torch.

[Fix] spell_is_inate() の未定義動作を修正

負のシフト量が発生しうるコードになっていた。
元のコードの意図は 32*3..32*4, 32*4..32*5, 32*5..32*6 の値を場合分けして
処理することだと思われるので、それに沿う形で修正。

To be more idiomatic, reword English message for genocide failing in an arena.

[Fix] X11 ドライバで日本語入力時にゴミが入ることがあるのを修正

例えば「ほげほげ」と入力すると後ろにゴミが入っていた。

原因は convert_to_euc() 内の buffer over-read。
convert_to_euc() は iconv() で変換されたバイト列が0終端されていると仮定
しているが、iconv() は0終端を行わないのでこれは正しくない。元のバッファ
サイズと iconv() が返した残りバッファサイズからサイズを計算するように修
正。

Simplify English message for no object at target location in fetch_item().

[Fix] X11 ドライバの buffer over-read を修正

Infofnt_text_std() に渡されたバイト列をバッファへコピーする際に
angband_strcpy() を使っているのが原因。
angband_strcpy() は入力文字列の長さを返すが、このとき strlen() 相当の処
理が行われる。しかし、Infofnt_text_std() に渡されるデータは term_win 構
造体内のバイト列であり、これは0終端されていない。よってこれを
angband_strcpy() に渡すのは誤り。

よって、単に memcpy() でコピーする。sbuf のサイズは十分だと仮定する。
もし len が sbuf のサイズより大きいと壊れるが、それは元のコードでも同じ
(iconv() に len を実質そのまま渡しているため)。

[Fix] X11 ドライバで Xft を有効にした際の未初期化メモリ参照を修正

valgrind による指摘。
maid-x11.c の create_pixel() において、XRenderColor の alpha メンバを初
期化していなかった。

[Fix] モンスターの打撃に対する思い出処理のエンバグを修正

increase_blow_type_seen() 内で monap_ptr の内容を参照している関係上、こ
の関数の呼び出しは命中/回避処理の後でなければならない。

合わせて、process_monster_blows(), increase_blow_type_seen() の仕様に関
するコメントを多少追加。

Merge pull request #72 from taotao54321/fix-kawarimi-message

[Fix] 変わり身失敗時のメッセージをわかりやすくした

[Fix] 変わり身失敗時のメッセージをわかりやすくした

変わり身の攻撃回避部分が失敗したときテレポートを試みるが、このときのメッセージが

* 成功時「失敗！攻撃を受けてしまった。」
* 失敗時「失敗！逃げられなかった。」

となっており、慣れていないと何が失敗したのかわかりにくい。
そこで、「変わり身失敗！〜」というメッセージに変更する。
英語メッセージも合わせて変更。

[fix] #65 未初期化の変数を参照している

魔法書のを選択する時にchoose_objectがNULLを返した場合、
item == INVEN_FORCEなら練気術が選択されたとするアドホックな
実装がなされているが、選択をキャンセルした場合もNULLが
返され、その場合itemは更新されていないので未初期化の変数itemが
たまたまINVEN_FORCEっだった場合にdo_cmd_mind()が呼ばれてしまう。

そもそもinventory_slot_typeにスロット非選択状態の値があるべきなので、
INVEN_NONEを新設して使用されていない値として1000を割り当てる。
choose_object()の最初で、第2引数に参照渡しされた変数をINVEN_NONEで
初期化するようにする。

Merge pull request #71 from dis-/feature/Refactor-Force-Sleep

[Refactor] FORCE_SLEEPという不適切なフラグ名称 #30

[Refactor] FORCE_SLEEPという不適切なフラグ名称 #30

FORCE_SLEEPというフラグ名を実態に合わせてPREVENT_SUDDEN_MAGICに変更。
r_info.txtのフラグ名も合わせて変更。
合わせてMFLAG_NICEも実態に合わせてMFLAG_PREVENT_MAGICに変更。
実動作の変更はなし。

[Fix] 打撃を受けた際の落馬/変わり身処理の発動条件を修正

落馬/変わり身処理は命中時のみ行うようにした。

落馬に関しては元々ダメージ0なら落馬しないという条件があるので実質変更なし。

変わり身に関しては、「〜の攻撃をかわした。失敗！攻撃を受けてしまった。」
というメッセージ表示が不自然であることからこのようにした。厳密には仕様変
更となるので、もし問題となるようなら仕様を再考することとしたい。

この修正に伴い、思い出処理を命中判定より先に行うようにした。これを後に置
くと変わり身発動時の return により飛ばされてしまう可能性があるため。
(思い出処理は命中/変わり身の成否によらず行うという従来の仕様を維持)

Merge pull request #67 from dis-/feature/Fix-Target-Clear

[Fix] 視界内のモンスター表示中、center_playerの描写がおかしくなる #66

[Fix] 視界内のモンスター表示中、center_playerの描写がおかしくなる #66

#38で実装したtarget_clear()内のverify_panel()が不要だった。
fix_monster_list()内で座標更新処理が行なわれていたため、描写が二重になっていた。
該当の処理を省くことで処理を正常化する。

Merge pull request #40 from hengband/feature/english-rumor

[Fix] 英語版 rumor.txt に有用な噂追加. / Add useful rumors to English version.

Merge pull request #62 from habu1010/feature/Buildtest-parallel-execution

[update] #1 ビルドテストを並列実行する