Matthias Melcher wrote:
>>      Hmm, you should only be including Fl_Native_File_Chooser.H,
>>      not Fl_Native_File_Chooser_FLTK.H.
> 
> Which reminds me: can't we move the content of all the 
> Fl_Native_File_Chooser_XXX.H
> files into the Fl_Native_File_Chooser.H file? 

        I investigated doing that while developing the code, but due to the
        platform-specific nature of the native file choosers, it involved having
        #ifdef's peppered throughout the class which I thought made it
        ugly and unreadable.

        FNFC is an oddball widget in that it is almost entirely
        platform specific code.

> This issue will keep repeating itself otherwise.

        It shouldn't repeat with the correction to the docs.
        The docs were pointing to the wrong file (see my followups).
        I was wondering why folks kept including the wrong file..!

        I don't think folks are perusing the include dir looking for files
        to include, they were just getting wrong info from the docs.

        Might be a good idea though to rename the Fl_Native_File_Chooser_XXX.H
        files to include a trailing underbar (eg. Fl_Browser_.H) to give a hint
        not to include them directly.

        But fixing the docs should definitely help..!
_______________________________________________
fltk mailing list
[email protected]
http://lists.easysw.com/mailman/listinfo/fltk

Reply via email to