Closed datasynergyuk closed 4 years ago
Please can I suggest updating the current bookmark index (m_iCurrent) when the bookmark manager is used to jump to a specific bookmark (CHexBookmarks::GoBookmark). This will make subsequent next/previous bookmark behaviour more consistent.
Yes, good catch. I've also disposed of IHexBkmVirtual::GetNext/GetPrev/Remove methods. I think all next/prev logic can be handled by the control now.
IHexBkmVirtual::GetNext/GetPrev/Remove
Please can I suggest updating the current bookmark index (m_iCurrent) when the bookmark manager is used to jump to a specific bookmark (CHexBookmarks::GoBookmark). This will make subsequent next/previous bookmark behaviour more consistent.