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

Dialog windows of modules crash application #1

Closed
dmarszk opened this issue Sep 19, 2015 · 7 comments
Closed

Dialog windows of modules crash application #1

dmarszk opened this issue Sep 19, 2015 · 7 comments

Comments

@dmarszk
Copy link
Collaborator

dmarszk commented Sep 19, 2015

According to Wisling, any child afx window opened by dialog window ends up crashing TA.

@dmarszk dmarszk added the bug label Nov 4, 2015
@dmarszk
Copy link
Collaborator Author

dmarszk commented Mar 31, 2016

I'm actually finding it hard to reproduce when using eaca85d as head. @Wisling please advise.

@Wisling
Copy link
Owner

Wisling commented Apr 5, 2016

I checked and it's only in the debug build of TA.

@dmarszk
Copy link
Collaborator Author

dmarszk commented Apr 5, 2016

Could you provide me with steps to reproduce?

2016-04-05 14:44 GMT+02:00 Wisling notifications@github.com:

I checked and it's only in the debug build of TA.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#1 (comment)

@Wisling
Copy link
Owner

Wisling commented Apr 5, 2016

Yeah, if you don't experience it, then let me know.
Build TA under Debug.
Run TA & connect to any Tibia session.
Go to spells, healing, and try and edit the exura sio Heal List. If there is an error before the window appears, then that's what I'm referring to.

@dmarszk
Copy link
Collaborator Author

dmarszk commented Apr 5, 2016

Thanks.
I found the culprit. All modules are being derived from CWinApp. It's pointless, because all modules are actually DLLs acting as hidden dialogs. CWinApp makes MFC think each module is a separate application, and tries to assign the child dialog boxes to it. I'll fix it after merging current patches.

@dmarszk
Copy link
Collaborator Author

dmarszk commented Apr 7, 2016

Fixed in d08e5d8

@dmarszk dmarszk closed this as completed Apr 7, 2016
@Wisling
Copy link
Owner

Wisling commented Apr 9, 2016

That probably explains why tabbing didn't work either. I may need to undo some work-arounds :P

Wisling referenced this issue Jul 18, 2016
Added -8 offset used for addrArrayPtrRecvStream into the code directly instead of adjusting the address in the xml each time
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants