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 グローバル変数も消せるはず。
今後のリファクタ課題。

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] 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 ビルドテストを並列実行する

Merge pull request #61 from backwardsEric/english-spell-edits

3.0.0.0: minor English edits for src/spell

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

GitHubホストランナーはマルチコアCPUなので、ビルドテストを
並列ビルドに対応させる。
nprocコマンドでCPUスレッド数が取得できるので、makeコマンドの
-jオプションにnprocコマンドの結果を与えるようにする。

[Fix] 有用な噂で得られる情報の可能性を日本語版と同じにする。 / English version makes the possibility of information obtained from useful rumors the same as the Japanese version.

 * ダンジョンは鉄獄のみ、他は全て知り得る可能性がある。

Since use a plural verb ("appear") with "group", emphasizing the indidviuals in the group, make the noun in "as your x" plural.

Merge pull request #58 from backwardsEric/english-realm-edits

3.0.0.0: minor English edits for src/realm

Merge pull request #59 from backwardsEric/english-room-edits

3.0.0.0: minor English edits for src/room

Merge pull request #60 from backwardsEric/english-specific-object-edits

3.0.0.0: minor English edits for src/specific-object

Change English message for attempt to fish without nearby water.

Change to past tense for English message after a successful life rate change.

In English message for toragoroshi activation, add "the" and replace "enchanted" with "fascinated".

In build_type17() English cheat message, replace "fix" with "fixed" and make capitalization like the other cheat messages.

Changed English cheat message for build_target_vault():  was "Elemental Vault" the same as for build_elemental_vault().

Change English cheat message for build_bubble_vault(): was "Room Vault", the same as for build_room_vault().

Reword English description of "Trump Kamikaze" to hopefully be less awkward.

Since "goddess" is singular, add "s" when forming possessive.

Insert indefinite article to be more idiomatic.

To be more idiomatic, make "wall" plural in the onset message for "Sound of disintegration".

Make "corpse" plural in the English descriptions for "Animate Dead" and "Firiel's Song".

To be more idiomatic, make "curse" plural in the descriptions for "Remove All Curse" and "Dispel Curse".

Reword English description of "Cure Poison": "poison status" didn't add much beyond what's conveyed by the spell title.

Reword English description of "Twin Slash".

For clarity, drop "already" in English description of the "Anti multiply barrier".

Swap "in" to "on" to be more idiomatic.

Insert missing "are".

Changed English title of demon spell from "Kiss of Succubus" to "Succubus's Kiss".

Change English title of demon spell from "The Flow of Lava" to "Lava Flow".

Fix English typo.

Change the English description of "Summon Manes" to reflect that a group is possible.

Reword English description of "Orb of Entropy" to be more idiomatic.