STR #3162: fl_file_chooser() breaks pixmap_browser.cxx on Win7

STR #3162

Application:	FLTK Library
Status:	1 - Closed w/Resolution
Priority:	4 - High, e.g. key functionality not working
Scope:	2 - Specific to an operating system
Subsystem:	None
Summary:	fl_file_chooser() breaks pixmap_browser.cxx on Win7
Version:	1.3.3
Created By:	szukw000
Assigned To:	AlbrechtS
Fix Version:	None
Update Notification:	Receive EMails Don't Receive EMails

Trouble Report Files:

Trouble Report Comments:

#1	szukw000 20:41 Nov 24, 2014

Well, I know that you never ever change files in the test directory. Nevertheless, I try. The 'fl_file_chooser()' does not work in Win7. I do not know why. But I know the 'Fl_Native_File_Chooser' does work on Win7. Therefore this patch. winfried

#2	AlbrechtS 07:07 Nov 25, 2014

Thanks for the proposal, but what is the problem with fl_file_chooser() on Windows 7? I don't know of any issues, and it works for me on Windows 8.1 (compiled with MinGW). Maybe you have some ABI problems? Please check your build: use a fresh checkout and don't set any ABI options, or do a fresh checkout, change the ABI options in FL/Enumerations.H (the officially supported way), and do a full build then. If any of these doesn't work, please provide more info (particularly error messages). We would rather fix it than replace the file chooser. That said: We _do_ change test/*.cxx files, but these files are primarily for function tests, and fl_file_chooser() should be included. Hence I believe the patch as such is not acceptable. There is also examples/nativefilechooser-simple-app.cxx to exercise the native file chooser. This is a place where patches and other suggestions are more appropriate. Maybe an option to switch the GTK file chooser on/off interactively would be interesting (if not yet included)... PS: please post patches as normal text files (*.patch or *.diff), at least if they are not oversized. The one you posted is less than 2 KB and should be posted unzipped, so that you can take a look at the patch w/o downloading and unzipping it. Thanks.

#3	szukw000 11:11 Nov 25, 2014

>Thanks for the proposal, but what is the problem with >fl_file_chooser() on Windows 7? I don't know of any >issues, and it works for me on Windows 8.1 (compiled with MinGW). I use MSVC 2010 Express with: Microsoft (R) C/C++ Optimizing Compiler Version 16.00.40219.01 for x64 Copyright (C) Microsoft Corporation. All rights reserved. In an application one year ago I changed from 'fl_file_chooser()' to 'Fl_Native_File_Chooser' because of a 'BEX64' bug report. This 'BEX64' remains to show up in 'pixmap_browser()' in Win7. But as 'fl_file_chooser()' does work on Windows 8.1 (compiled with MinGW): remove this report. winfried

#4	AlbrechtS 08:49 Nov 26, 2014

I googled for BEX64, and this seems to be caused by a corrupted file or dll problem, maybe related to Internet Explorer or one or more of its dlls. Since we didn't get other reports about broken fl_file_chooser() this is probably a problem on the OP's system. Closing STR on OP's request.