Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"The Thinking Man's Thesaurus" Error When Launching Works When Ignoring (Recently Cropped-Up) #1124

Closed
alienfxfiend opened this issue Feb 16, 2022 · 6 comments

Comments

@alienfxfiend
Copy link

ththes13.zip

I get an 'Interrupt' error recently when opening "The Thinking Man's Thesaurus" —it was working perfectly before yesterday when I got some Cumulative Windows 10 update & a .NET 5 update (maybe that's the 'cause ?) […] I don't understand —can it be fixed please ?! When I click continue it loads normally (I noticed the cipboard gets auto-pasted maybe it corrupts the app ?), also it only occurs /w the taskbar-pinned shortcut (/w a custom icon) not when I run THES.EXE via a Launcher app ("Everything" by voidtools), I'm thinking maybe it's a prob /w my .ini file ? I opened it & Use Single Processor was disabled, I had it use only one Processor before to avoid errors. It was working w/o a hitch a few days ago, I had enabled HIPS (Host Intrusion Protection System) on Comodo Internet Security Premium but now I have it disabled, so it couldn't be that —I get the latest version from AppVeyor.

P.S: It's a tiny app /w no setup runs directly /w THES.EXE.

@alienfxfiend
Copy link
Author

Nevermind "ForceSingleProcessor=1" somehow got disabled again, I will test again & see if it's resolved. Sorry for taking your time.

@alienfxfiend
Copy link
Author

The new OTVDM Heap corruption in OLE certainly improves loadtimes as that of Win 3.11 under DOSBox but sometimes I get an interrupt error (the app automatically enters clipboard text on loading) & sometimes it causes problems —please try to fix the issue, I get a block symbol in the apps input field, & sometimes this causes an error as of the latest commit (OLE Heap corruption) ! Collins Dictionary works perfectly though, only not this Thinking Man's Thesaurus !

@alienfxfiend alienfxfiend reopened this Mar 8, 2022
@cracyc
Copy link
Contributor

cracyc commented Mar 8, 2022

Need more detail. What is on the clipboard when it crashes? Did this start after the recent commits? Also a trace could help.

@alienfxfiend
Copy link
Author

alienfxfiend commented Mar 8, 2022

Bingo ! I found out what clipboard text was crashing the app (the aforementioned Interrupt crash.) It was an EmDash "—" (without quotes) ! Clicking Continue lets the app work normally though. I'm not certain if it was after the recent commit, the launchtime has certainly improved (Heap Corruption in OLE commit —which I got through AppVeyor.) On Windows 3.11 on DOSBox Games Launcher (frontend) the app opens instantly, & it does so now /w the recent commit too, which is great ! P.S: This error is not the same as the error I reported at the beginning of this Bug Report, this one is just when the EmDash character is in clipboard. The one I initially reported was a false flag because I had ForceSingleProcessor unchecked, since I didn't realize the zip overwrote the OTVDM.ini. Hopefully you can fix this little bug so it goes away.

@cracyc
Copy link
Contributor

cracyc commented Mar 8, 2022

Not fixable because it's a bug in the program as emdash is ascii 0x97 and it sign extends it to 0xff97 which causes a pointer underflow. Also reproducible in ntvdm. You can try this spell.dll replacement which might fix it
SPELL.zip
.

@alienfxfiend
Copy link
Author

alienfxfiend commented Mar 8, 2022

Thank you for clarifying —I will check it out pronto ! P.S: That works, it's fixed now ! ♥

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants