search

gsass1 / NTop

💻 htop-like system-monitor for Windows with Vi-keybindings.
GNU General Public License v3.0
1.23k stars 69 forks source link

Sort by memory is not sorted correctly #54

Open Shiratsui opened 1 year ago

Shiratsui commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

Hello @gsass1, any update?

cobrce commented 6 months ago

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