Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sort by memory is not sorted correctly #54

Open
Shiratsui opened this issue Jan 29, 2023 · 3 comments
Open

Sort by memory is not sorted correctly #54

Shiratsui opened this issue Jan 29, 2023 · 3 comments

Comments

@Shiratsui
Copy link

I have processes that have high memory usage
1
Image above shown using "-s mem"

but on the bottom of those processes, I have 2 high mem usage that did not got sorted
2

@gsass1
Copy link
Owner

gsass1 commented Jan 30, 2023

Hello, thanks for reporting!

I sadly don't have access to a Windows machine right now, so I can't really diagnose the issue.

In the next month I will probably be able to afford a machine that will handle a decently sized Windows VM for development though. Then I will update this project, or maybe even rewrite it in something that's not C.

@pidgeon777
Copy link

Hello @gsass1, any update?

@cobrce
Copy link
Contributor

cobrce commented May 17, 2024

Hello @gsass1,
SortProcessByUsedMemory is supposed to use __int64 to compare two "UsedMemory" but the generated code uses dword
image

Update:
I recompiled with VS community 2022 and it's fixed

image
Above 4 gigs process is on top
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants