https://bugs.kde.org/show_bug.cgi?id=471807

Aaron Merey <ame...@redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #160848|0                           |1
        is obsolete|                            |

--- Comment #12 from Aaron Merey <ame...@redhat.com> ---
Created attachment 160996
  --> https://bugs.kde.org/attachment.cgi?id=160996&action=edit
patch v6

(In reply to Mark Wielaard from comment #11)
> The move_DebugInfo_one_step_forward in VG_(load_di) and VG_(addr_load_di)
> seem unnecessary, but mostly harmless.
> Those are optimizations for VG_(find_DebugInfo) and find_DiCfSI, should
> probably only be done there.
> [...] 
> This looks good. I think we should remove those two
> move_DebugInfo_one_step_forward calls. I like to commit this with that small
> change so we can test it a bit more widely.

Thanks for all of these reviews. I've attached patch v6 with the
move_DebugInfo_one_step_forward calls removed.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to