mIRC Home    About    Download    Register    News    Help

Print Thread
dialog.mdx last beta 7.64.5829 #268542 10/03/21 05:17 PM
Joined: Jul 2014
Posts: 226
TECO Offline OP
Fjord artisan
OP Offline
Fjord artisan
Joined: Jul 2014
Posts: 226
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
}


TECO
irc.PTirc.org (Co-Admin)
Re: dialog.mdx last beta 7.64.5829 [Re: TECO] #268544 10/03/21 06:01 PM
Joined: Sep 2015
Posts: 97
klez Offline
Babel fish
Offline
Babel fish
Joined: Sep 2015
Posts: 97
Originally Posted by TECO
dialog.mdx dll


Switch to DCX Tools.. is more powerful than MDX laugh

Re: dialog.mdx last beta 7.64.5829 [Re: TECO] #268546 10/03/21 06:40 PM
Joined: Dec 2002
Posts: 4,914
Khaled Offline
Hoopy frood
Offline
Hoopy frood
Joined: Dec 2002
Posts: 4,914
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?

Re: dialog.mdx last beta 7.64.5829 [Re: Khaled] #268548 10/03/21 06:53 PM
Joined: Jul 2014
Posts: 226
TECO Offline OP
Fjord artisan
OP Offline
Fjord artisan
Joined: Jul 2014
Posts: 226
Can you get me the beta version 5513 back to confirm?


TECO
irc.PTirc.org (Co-Admin)
Re: dialog.mdx last beta 7.64.5829 [Re: TECO] #268550 11/03/21 04:25 AM
Joined: Jul 2014
Posts: 226
TECO Offline OP
Fjord artisan
OP Offline
Fjord artisan
Joined: Jul 2014
Posts: 226
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.


TECO
irc.PTirc.org (Co-Admin)
Re: dialog.mdx last beta 7.64.5829 [Re: TECO] #268568 15/03/21 10:11 AM
Joined: Dec 2002
Posts: 4,914
Khaled Offline
Hoopy frood
Offline
Hoopy frood
Joined: Dec 2002
Posts: 4,914
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.

Re: dialog.mdx last beta 7.64.5829 [Re: Khaled] #268569 15/03/21 03:29 PM
Joined: Jul 2014
Posts: 226
TECO Offline OP
Fjord artisan
OP Offline
Fjord artisan
Joined: Jul 2014
Posts: 226
Ok.
Thanks wink


TECO
irc.PTirc.org (Co-Admin)
Re: dialog.mdx last beta 7.64.5829 [Re: TECO] #268575 17/03/21 10:48 AM
Joined: Dec 2002
Posts: 4,914
Khaled Offline
Hoopy frood
Offline
Hoopy frood
Joined: Dec 2002
Posts: 4,914
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 these windows, so it cannot update the windows regions itself, and MDX is not setting the windows 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.

Re: dialog.mdx last beta 7.64.5829 [Re: Khaled] #268578 17/03/21 03:45 PM
Joined: Jul 2014
Posts: 226
TECO Offline OP
Fjord artisan
OP Offline
Fjord artisan
Joined: Jul 2014
Posts: 226
Thank you Khaled wink
The latest beta version v7.64.6087 left everything working properly.

Thank you very much wink

Last edited by TECO; 17/03/21 06:52 PM.

TECO
irc.PTirc.org (Co-Admin)