| Revision | d7b14f20b2449a81d7168eec62a78ee7922a7c28 (tree) |
|---|---|
| Time | 2018-09-21 23:00:41 |
| Author | Habu <habu@user...> |
| Commiter | Habu |
[fix]鍛冶師で武器/防具の強化ができない問題を修正
Warning潰しで配列を初期化するようにした事で、以前からあった潜在的バグが表面化した。
初期化するようにする前のコードでは、ループのスコープで宣言した未初期化変数は、
前回のループの内容を保持している事を前提としたコードになっていたが、
当然ながらC言語の仕様にそのような保証は無く、たまたま正常に動いていたに過ぎない。
変数宣言をループの外に出すことで正常に動作するように修正。
src/object2.c (diff)| @@ -8106,6 +8106,7 @@ static void add_essence(ESSENCE_IDX mode) | ||
| 8106 | 8106 | char o_name[MAX_NLEN]; |
| 8107 | 8107 | int use_essence; |
| 8108 | 8108 | essence_type *es_ptr; |
| 8109 | + bool able[22] = { 0 }; | |
| 8109 | 8110 | |
| 8110 | 8111 | int menu_line = (use_menu ? 1 : 0); |
| 8111 | 8112 |
| @@ -8138,7 +8139,6 @@ static void add_essence(ESSENCE_IDX mode) | ||
| 8138 | 8139 | choice = (always_show_list || use_menu) ? ESCAPE:1; |
| 8139 | 8140 | while (!flag) |
| 8140 | 8141 | { |
| 8141 | - bool able[22] = {0}; | |
| 8142 | 8142 | if( choice==ESCAPE ) choice = ' '; |
| 8143 | 8143 | else if( !get_com(out_val, &choice, FALSE) )break; |
| 8144 | 8144 |
Fork