mIRC Homepage
Posted By: TECO dialog.mdx last beta 7.64.5829 - 10/03/21 05:17 PM
Hi Khaled,

The dialog.mdx dll file started to have problems with this latest beta update.
In the code that I present below, the code does not appear in a normal way. Perhaps there is nothing to be done about this problem, or if there is, it would be good.

mIRC 7.64
[Linked Image from i.ibb.co]

mIRC 7.64.5829
[Linked Image from i.ibb.co]

Code:
Code
mdxfile { return $+($scriptdir,dlls\,$1,.mdx) }
prevbmp {
  if ($dialog($1)) {
    var %w = $+(@sprev.,$1,.,$2)
    if (!$window(%w)) { ppwin $1- }
    drawrect -nf %w $color(back) 1 0 0 $window(%w).w $window(%w).h
    if ($6- != $null) { drawtext -npb %w $color($4) $color(back) $window($active).font 13 $5 1 $6- }
    drawdot %w
  }
}
ppwin {
  mdx SetControlMDX $1-2 Window > $mdxfile(dialog)
  var %w = $+(@sprev.,$1,.,$2)
  window -phf %w -1 -1 $window(-1).w $window(-1).h
  did -a $1-2 grab $window(%w).hwnd %w
}
Posted By: klez Re: dialog.mdx last beta 7.64.5829 - 10/03/21 06:01 PM
Originally Posted by TECO
dialog.mdx dll


Switch to DCX Tools.. is more powerful than MDX laugh
Posted By: Khaled Re: dialog.mdx last beta 7.64.5829 - 10/03/21 06:40 PM
My reply is going to be the same as the one here.

That said, there were very few changes in the latest beta and I cannot see anything in the beta that would cause such an issue.

Are you sure that this issue began with 5829?
Posted By: TECO Re: dialog.mdx last beta 7.64.5829 - 10/03/21 06:53 PM
Can you get me the beta version 5513 back to confirm?
Posted By: TECO Re: dialog.mdx last beta 7.64.5829 - 11/03/21 04:25 AM
Hi Khaled
In beta version 7.64.5513 it works correctly.

[Linked Image from i.ibb.co]


Thank you maroon for sending me all the beta versions to test.
Posted By: Khaled Re: dialog.mdx last beta 7.64.5829 - 15/03/21 10:11 AM
Thanks for tracking this down. I have looked at the code changes between beta 5513 and 5829 and there is one change, relating to window regions and the MDI theme code, which is likely causing this. However, the change is intentional and I shouldn't really change it back to how it was in 5513. It may be that MDX/DCX are accessing custom @window regions in a way that is affected by timing or windows messages. It's difficult to say without looking at their code. I am going to make a change to the order in which regions are set in the next beta to see if that helps.
Posted By: TECO Re: dialog.mdx last beta 7.64.5829 - 15/03/21 03:29 PM
Ok.
Thanks wink
Posted By: Khaled Re: dialog.mdx last beta 7.64.5829 - 17/03/21 10:48 AM
Thanks for the test scripts. It looks like the issue is indeed related to windows regions. MDX appears to be creating/redirecting/hooking windows/messages in order to capture the output of the /draw commands. This results in mIRC not receiving sizing messages for the window, so it cannot update the window region itself, and MDX is not setting the window region appropriately (most likely because, in previous versions, this was not needed).

The way beta 5829 handles this is technically correct. However, I am going to make a change that should resolve the issue with MDX (and possibly the DCX issue reported in another thread). This change should be in the next beta.
Posted By: TECO Re: dialog.mdx last beta 7.64.5829 - 17/03/21 03:45 PM
Thank you Khaled wink
The latest beta version v7.64.6087 left everything working properly.

Thank you very much wink
© mIRC Discussion Forums