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

Crash on Add New Source File in IDE in Windows for Workgroups 3.11 #1267

Open
nickkeane opened this issue Apr 7, 2024 · 3 comments
Open
Labels

Comments

@nickkeane
Copy link

The IDE crashes when I create a new source file in my installation of Open Watcom in 86Box. When I create a project, set target environment to Win16 and image type as Executable, it creates the project. Then, when clicking Sources -> New Source and entering "main.c" and clicking Add or pressing the Enter key, the IDE crashes.

https://pasteboard.co/hog7EMkxgSPB.png

@jmalak jmalak added the bug label Apr 7, 2024
@jmalak
Copy link
Member

jmalak commented Apr 7, 2024

Thank you for your bug report.

@Baron-von-Riedesel
Copy link

Baron-von-Riedesel commented Apr 14, 2024

I confirm this crash - actually, it's a GPF in the IDE, file vfilelst.cpp, function AddSrcDlgProc(). Register DS contains a selector of Win16 dll COMMDLG, which almost certainly is wrong - since it is a so-called callback, register DS "should" be set on function entry, which obviously isn't done.

Comparing ide.exe with OW v1.9, OW20's ide has just 11 exports, while OW19's has 15 - and the export for AddSrcDlgProc() is one of those that are missing.

@jmalak
Copy link
Member

jmalak commented Apr 14, 2024

@Baron-von-Riedesel thanks for your analysis. It is missing export modifier on callback function for 16-bit widows.
The main difference between OW 1.9 and OW 2.0 sources for 16-bit Windows that callback functions in OW 2.0 must be explicitly exported against OW 1.9 where is implicitly exported. Most of callback functions were fixed, but some is still incorrect (no exported).
I will check and add missing export modifier for appropriate callback 16-bit Windows function.

jmalak added a commit that referenced this issue Apr 14, 2024
missing export modifier for 16-bit Windows AddSrcDlgProc callback funstion.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants