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.

Sign up for GitHub

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

Jump to bottom

Drag & drop file into Notepad2 cause the file to be loaded twice #703

Closed
zufuliu opened this issue Aug 13, 2023 · 1 comment
Closed

Drag & drop file into Notepad2 cause the file to be loaded twice #703

zufuliu opened this issue Aug 13, 2023 · 1 comment
Labels
💪help wanted
Milestone
v4.23.08

Comments

@zufuliu
Copy link
Owner

zufuliu commented Aug 13, 2023

Following code is intended to fix the bug when drag & drop file from 32 bit app to 64 bit Notepad2 (see https://stackoverflow.com/questions/39612616/drag-and-drop-from-32-to-64-bit/39664759#39664759):
https://github.com/zufuliu/notepad2/blob/361dfdee4dbb24965e01c7811062f4aee14c7279/scintilla/win32/ScintillaWin.cxx#L3887-L3888

However this cause (tested on Win 10) main windows receives two WM_DROPFILES messages (with identical wParam and lParam).
https://github.com/zufuliu/notepad2/blob/361dfdee4dbb24965e01c7811062f4aee14c7279/src/Notepad2.c#L1241-L1244
@zufuliu zufuliu added this to the v4.23.08 milestone Aug 14, 2023
@zufuliu zufuliu removed the 🐛bug label Aug 14, 2023
zufuliu added a commit that referenced this issue Aug 14, 2023
@zufuliu
Fix file been loaded twice after dropped onto Notepad2, issue #703.
3fb3db6
@zufuliu
Copy link
Owner Author

zufuliu commented Aug 14, 2023

Fixed by 3fb3db6.

@zufuliu zufuliu closed this as completed Aug 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
💪help wanted
Projects
None yet
Milestone
v4.23.08
Development

No branches or pull requests
1 participant
@zufuliu