Page 5 of 5

Re: Bug in Multiview Search requester

Posted: Thu Nov 02, 2017 11:28 am
by salass00
As far as corrupted characters popping up in the string gadgets of the ASL requesters this is IMHO probably due to the way that asl.library interacts with the string gadgets and is IIRC not something that is easily fixed. Either any code that modifies the string contents should be made to never leave the string in a state without proper nul-termination (might involve using Forbid()) or there needs to be some way to read the contents of the string gadget with proper locking (STRINGA_TextVal is AFAICT not really safe to read and use while the string is might be being edited).

Fortunately for the MultiView search requester these problems should not apply there because it only needs to read the string value after the window has been closed.

Re: Bug in Multiview Search requester

Posted: Thu Nov 02, 2017 3:52 pm
by xenic
Raziel wrote: I can't even find such an option to patch requesters, so i believe i'm not using it.
You would know it if your ASL is patched; the patched ASL requester is obviously an MUI window. It's not an MUI preferences item. If you install MUI with the installer in the MUI directory (MUI/Install-MUI) there is an option to add the patch to your user-startup. It looks like this:

;BEGIN MUI ASL
if exists "SYS:MUI"
run <>nil: MUI:PatchASL
endif
;END MUI ASL

It runs the PatchASL utility in the MUI directory. If you install MUI with AutoInstall, I don't think you get the patch option for some reason.

EDIT: Now that I think of it, you might want to try the MUI ASL patch to see if your problem occurs with the MUI ASL requester too. Just enter "MUI:PatchASL" in a shell before conducting your test. Of course you'll need to reboot to get rid of the patch after testing.

Re: Bug in Multiview Search requester

Posted: Thu Nov 02, 2017 9:39 pm
by Raziel
xenic wrote:
Raziel wrote: I can't even find such an option to patch requesters, so i believe i'm not using it.
You would know it if your ASL is patched; the patched ASL requester is obviously an MUI window. It's not an MUI preferences item. If you install MUI with the installer in the MUI directory (MUI/Install-MUI) there is an option to add the patch to your user-startup. It looks like this:

;BEGIN MUI ASL
if exists "SYS:MUI"
run <>nil: MUI:PatchASL
endif
;END MUI ASL

It runs the PatchASL utility in the MUI directory. If you install MUI with AutoInstall, I don't think you get the patch option for some reason.

EDIT: Now that I think of it, you might want to try the MUI ASL patch to see if your problem occurs with the MUI ASL requester too. Just enter "MUI:PatchASL" in a shell before conducting your test. Of course you'll need to reboot to get rid of the patch after testing.
Nope, no patch installed here.

And i think i skip the testing of third party patches for now, until there's a solution for this bug.

But thank you for the hints

Re: Bug in Multiview Search requester

Posted: Fri Jun 08, 2018 4:03 pm
by Raziel
Just wanted to ask for an update on this issue as it is biting me now every day...not only in Multiview/Multiviewer/Notepad/MultiEdit, but also, and for some time now, pestering me with simple copy and paste actions in WB/shell.

It goes as far as copying a string in one shell window, pasteing it into another and getting a string of unreadable characters and a crashing app.

This is annyoing as hell and makes "working" with the Amiga a PITA.

Any time estimate when this patch will hit the streets