Optimize udmpparser::UserDumpParser::GetMemBlock

[0-x-2-2]

It works in my testing, let me know if you can find any issues with it. This is significantly faster, has worked in my tooling, and has sped up IAT resolving from 5028ms to 24ms on my machine in my application when profiled.

[0vercl0k]

Oof great find - thank you for diagnosing the issue, taking the time to send a PR 🙏🏽 (and sorry for introducing this bug 🫢)

Let me work off this PR to fix the CI & update how the Python wheels gets generated.. shouldn't take me too long.

Cheers

[0vercl0k]

Actually, isn't this a bug if (it != Mem_.begin()) { ? Shouldn't it be Mem_.end() @0-x-2-2 or I am missing something?

[0vercl0k]

Okay never mind, I read that piece of code too fast - my apologies 😅

Cheers

[0vercl0k]

All right, if this looks good to you I'll get this merged this weekend 🙂 thank you again!

Cheers

[0-x-2-2]

That sounds good. Honestly, you most likely spent more time reading the code than it took me to write it.

[0vercl0k]

Haha awesome - to be fair, you diagnosed the problem as well :)

In any case, thank you again; let's merge this and will keep the thread updated when the Python wheels are up.

Cheers

[0vercl0k]

Okay, v0.6 has been published on pypi:

>pip install udmp_parser
Requirement already satisfied: udmp_parser in c:\users\over\anaconda3\lib\site-packages (0.6.0)

over@panther:~$ pip install udmp_parser
Defaulting to user installation because normal site-packages is not writeable
Collecting udmp_parser
  Downloading udmp_parser-0.6.0-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (10 kB)
Downloading udmp_parser-0.6.0-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (235 kB)
Installing collected packages: udmp_parser
Successfully installed udmp_parser-0.6.0

[0-x-2-2]

Thanks.