Author Topic: MemStatTray UASM x64  (Read 6194 times)

fearless

  • Member
  • ****
  • Posts: 573
    • Github
MemStatTray UASM x64
« on: July 14, 2019, 06:45:43 AM »
I created an x64 version of the MemStatTray demo with UASM, to compliment the x86 version in the other topic
fearless

ASUS Crosshair 8 Hero, AMD 5950X, 32GB, MSI 5700XT, NZXT Kraken Z73, Seasonic 1000W PSU

Github Twitter Mastodon Gitbook

hutch--

  • Administrator
  • Member
  • ******
  • Posts: 10572
  • Mnemonic Driven API Grinder
    • The MASM32 SDK
Re: MemStatTray UASM x64
« Reply #1 on: July 14, 2019, 09:47:31 AM »
Just tested the file, works fine.  :thumbsup:
hutch at movsd dot com
http://www.masm32.com    :biggrin:  :skrewy:

LiaoMi

  • Member
  • *****
  • Posts: 1051
Re: MemStatTray UASM x64
« Reply #2 on: July 18, 2019, 07:13:23 AM »
I created an x64 version of the MemStatTray demo with UASM, to compliment the x86 version in the other topic

Hi fearless,

after three days of intensive use, I noticed that the numbers in the tray no longer change, it's hard to say what influenced this behavior, even if I reset all memory to 30 percent, nothing changes. Any suggestions?

fearless

  • Member
  • ****
  • Posts: 573
    • Github
Re: MemStatTray UASM x64
« Reply #3 on: July 18, 2019, 08:59:11 AM »
Might be a gdi leak somewhere, or a memory leak with constantly setting the shellnotify stuff maybe?
fearless

ASUS Crosshair 8 Hero, AMD 5950X, 32GB, MSI 5700XT, NZXT Kraken Z73, Seasonic 1000W PSU

Github Twitter Mastodon Gitbook

aw27

  • Guest
Re: MemStatTray UASM x64
« Reply #4 on: July 19, 2019, 06:59:26 AM »
Invoke MUITrayMenuSetTrayIconText, hTrayMenu, Addr szMemLoadPercent, NULL, MUI_RGBCOLOR(96,176,207)
mov hTMIcon, rax ; save returned icon handle for later

does not check for errors coming from MUITrayMenuSetTrayIconText, when it returns NULL in 2 instances.

fearless

  • Member
  • ****
  • Posts: 573
    • Github
Re: MemStatTray UASM x64
« Reply #5 on: July 19, 2019, 07:04:02 AM »
Ok, thanks for looking into it
fearless

ASUS Crosshair 8 Hero, AMD 5950X, 32GB, MSI 5700XT, NZXT Kraken Z73, Seasonic 1000W PSU

Github Twitter Mastodon Gitbook

aw27

  • Guest
Re: MemStatTray UASM x64
« Reply #6 on: July 20, 2019, 12:17:12 AM »
Ok, thanks for looking into it
I wonder why you did not provide the 64-bit versions of the ModernUI libraries, including source code, required to build your program. A possible cause for the bug is a defective translation to 64-bit.

LiaoMi

  • Member
  • *****
  • Posts: 1051
Re: MemStatTray UASM x64
« Reply #7 on: July 20, 2019, 01:24:28 AM »
I have repeated the bug three times, unfortunately the attachment through debugger does not work  :sad:

fearless

  • Member
  • ****
  • Posts: 573
    • Github
Re: MemStatTray UASM x64
« Reply #8 on: July 20, 2019, 07:38:03 AM »
I wonder why you did not provide the 64-bit versions of the ModernUI libraries, including source code, required to build your program
The libraries including source code are always available at the github repository: https://github.com/mrfearless/ModernUI and https://github.com/mrfearless/ModernUI64

I didn't include them mainly due to lazyness.

A possible cause for the bug is a defective translation to 64-bit.
Yes its possible there is some bugs in the porting process - I will try and look into it when I get a chance - the x64 version of the libraries are always a little bit behind and any fixes in the x86 version might not have been implemented in the x64 version - or I overlooked/forgot to do them.
fearless

ASUS Crosshair 8 Hero, AMD 5950X, 32GB, MSI 5700XT, NZXT Kraken Z73, Seasonic 1000W PSU

Github Twitter Mastodon Gitbook

aw27

  • Guest
Re: MemStatTray UASM x64
« Reply #9 on: July 20, 2019, 04:34:02 PM »
Quote
The libraries including source code are always available at the github repository: https://github.com/mrfearless/ModernUI and https://github.com/mrfearless/ModernUI64

I have not seen it because it is not pinned in https://github.com/mrfearless, I should have looked better.

Anyway, it is always a PITA to build when we are dealing with libraries we are not used to, then fix the UASM include files which were never properly updated to 64-bit then look for the definition of some macros that appear out of the blue. This does not apply specifically to the MemStatTrayx64 or to your good work in general.


hutch--

  • Administrator
  • Member
  • ******
  • Posts: 10572
  • Mnemonic Driven API Grinder
    • The MASM32 SDK
Re: MemStatTray UASM x64
« Reply #10 on: July 21, 2019, 02:56:23 AM »
I split the topic as I did not want fearless's thread to be ruined by arguments. The rest of the topic is where its easy enough to argue in without wrecking this topic.
hutch at movsd dot com
http://www.masm32.com    :biggrin:  :skrewy: