diff options
author | Yusuke Endoh <mame@ruby-lang.org> | 2022-07-11 14:53:12 +0900 |
---|---|---|
committer | Yusuke Endoh <mame@ruby-lang.org> | 2022-07-11 23:38:37 +0900 |
commit | a871fc4d86e857b4c580604d2654877135484896 (patch) | |
tree | 1a56b2f6ff0f108f13867ac0877de071991c6c50 /compile.c | |
parent | 2733c049674298cbc2130689a0a40013f3458755 (diff) |
Fix a regression of b2e58b02aec73f9c350bf109c021c180fc699ccc
At that commit, I fixed a wrong conditional expression that was always
true. However, that seemed to have caused a regression. [Bug #18906]
This change removes the condition to make the code always enabled.
It had been enabled until that commit, albeit unintentionally, and even
if it is enabled it only consumes a tiny bit of memory, so I believe it
is harmless. [Bug #18906]
Notes
Notes:
Merged: https://github.com/ruby/ruby/pull/6112
Diffstat (limited to 'compile.c')
-rw-r--r-- | compile.c | 11 |
1 files changed, 4 insertions, 7 deletions
@@ -2226,13 +2226,10 @@ static int add_adjust_info(struct iseq_insn_info_entry *insns_info, unsigned int *positions, int insns_info_index, int code_index, const ADJUST *adjust) { - if (insns_info[insns_info_index-1].line_no != adjust->line_no) { - insns_info[insns_info_index].line_no = adjust->line_no; - insns_info[insns_info_index].events = 0; - positions[insns_info_index] = code_index; - return TRUE; - } - return FALSE; + insns_info[insns_info_index].line_no = adjust->line_no; + insns_info[insns_info_index].events = 0; + positions[insns_info_index] = code_index; + return TRUE; } /** |