キーワードを指定してタグジャンプができないバグを修正

[7-rate]

PR の目的

#836 にてデグレしていたため、修正する。
通常のタグジャンプは可能だが、キーワードを指定してタグジャンプ機能ができなくなっていた。

カテゴリ

• 不具合修正

PR の背景

#836 で、タグジャンプを高速化するためにtagsファイルのパラメータ(!_TAG_FILE_FORMAT等)の読み込みのための関数ReadTagsParameterを作成しました。

タグジャンプの際にはReadTagsParameter()の後にfind_key_for_BinarySearch() または find_key_for_LinearSearch()にてタグを検索する作りにしていましたが、
find_key_for_LinearSearch()のときに今回のバグが再現します。
→キーワードを指定してタグジャンプの時はfind_key_for_LinearSearch()になりますので、タグジャンプできない状態でした。

細かい話としては、ReadTagsParameter()を抜ける際にはファイルシーク位置はtagsファイルのパラメーター群を読み終わった所にしたいため、old_offsetにて読み込んだ位置を保持しているつもりでした。
しかし、old_offsetの設定は到達不可能な場所で行っていたため、ReadTagsParameter()を抜ける際のファイルシークは初期化していないold_offset(不定値)での位置になっていました。

PR のメリット

キーワードを指定してタグジャンプ機能が従来通り使えるようになります。

PR のデメリット (トレードオフとかあれば)

特になし

PR の影響範囲

タグジャンプ機能のみ

関連チケット

#836

参考資料

なし

[AppVeyorBot]

✅ Build sakura 1.0.2639 completed (commit 7bae8237d2 by @7-rate)

[AppVeyorBot]

✅ Build sakura 1.0.2640 completed (commit 9e4e97f193 by @7-rate)

[AppVeyorBot]

✅ Build sakura 1.0.2641 completed (commit 9da407a215 by @7-rate)

[berryzplus]

動作確認まで出来てないんですが、コードの範疇で問題なさそうなんでapproveにしてしまいました:smiley:

[berryzplus]

誰かマージの代行を頼みます。

[KENCHjp]

マージしやした。