FLTK logo

Re: [fltk/fltk] scandir_posix.c readentry should check for ENOENT (#238)

FLTK matrix user chat room
(using Element browser app)   FLTK gitter user chat room   GitHub FLTK Project   FLTK News RSS Feed  
  FLTK Apps      FLTK Library      Forums      Links     Login 
 All Forums  |  Back to fltk.issues  ]
 
Previous Message ]New Message | Reply ]Next Message ]

Re: [fltk/fltk] scandir_posix.c readentry should check for ENOENT (#238) Albrecht Schlosser Jun 16, 2021  
 

@michaelbaeuerle Thanks, Michael, for chiming in. I agree with your arguments and I'm looking further into the details for a possible solution.

it is documented that a POSIX conformant readdir() is required

I don't see this statement anywhere in the code and comments, but I agree that this is the core of the problem because the DOS readdir() function is obviously not POSIX compliant.

[OTOH: the reason to provide fl_scandir() is to support systems that lack a POSIX scandir() function. Isn't it to be expected that readdir() is not POSIX compliant on such systems as well?]

If readdir() is broken on such a platform, a POSIX conformant readdir() replacement
or a separate implementation for scandir(), e.g. in "scandir_dos.c", should be provided.

Nice idea; in general I agree. The problem with this (or any other) approach is how to determine the "broken platform" and how to construct the build system to use the proper replacement function. If this is possible we could try to do it, however...

"Plan B" would be to ask the OP to patch FLTK for his (unsupported, broken) platform and don't change the library code at all. This doesn't mean that this is my final vote, but unless we can find a good way this would be the last resort.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Direct Link to Message ]
 
     
Previous Message ]New Message | Reply ]Next Message ]
 
 

Comments are owned by the poster. All other content is copyright 1998-2024 by Bill Spitzak and others. This project is hosted by The FLTK Team. Please report site problems to 'erco@seriss.com'.