FLTK logo

Re: thread drift Re: [fltk.general] [Fl_Flow] A new layout managerfor FLTK

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.general  ]
 
Previous Message ]New Message | Reply ]Next Message ]

Re: thread drift Re: [Fl_Flow] A new layout managerfor FLTK Ian MacArthur Nov 25, 2022  
 
On Wednesday, 23 November 2022 at 18:17:08 UTC gggg... wrote:

Originally include/FL/Fl_Flow.H was checked in, but that file is generated from the .h files in src/Fl_Flow , as you point out.There’s also a Fl_Flow.h (note the capitalization) in that folder. This caused me lots of confusion while I was developing this feature. I kept making changes in Fl_Flow.H only to have them disappear the next time I ran make. IMHO, generated files should never be checked in, but ultimately this is up to Karsten, of course. I was somewhat surprised that he accepted that change without comment.


Way off-topic, but from a portability perspective having filenames (auto-generated or otherwise) that differ only in name-case is just a disaster wating to happen, really.
It would probably be safer and more portable to make the auto-generated file have some unique name, "Just In Case"... say Fl_Flow_hdr.H or etc., just to ensure it can not "collide" with the "other" Fl_flow.h file...
 

--
You received this message because you are subscribed to the Google Groups "fltk.general" group.
To unsubscribe from this group and stop receiving emails from it, send an email to fltkgeneral+unsubscribe@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/fltkgeneral/5d9b8469-1410-4378-ab84-c237e9a31340n%40googlegroups.com.
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'.