Re: forward declarations patch

2000-07-19 Thread Jean-Marc Lasgouttes
> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: Angus> Because unnecessary inclusion of header files in header files Angus> is evil? Actually, it means that Dialogs.C no longer depends on Angus> form_xxx.h, so you can play with the forms and not have to Angus> recompile as much code. L

Re: forward declarations patch

2000-07-19 Thread Juergen Vigna
On 19-Jul-2000 Jean-Marc Lasgouttes wrote: > Angus> All files in the src/frontends/xforms/ directory are now self > Angus> consistent in this regard. > > Hmm, why is this situation better? Not that I am against the patch, I > just want to know. The situation is better because if you modify the

Re: forward declarations patch

2000-07-19 Thread Angus Leeming
JMarc> Angus> Attached is a small patch to FormPreferences and FormPrint, JMarc> Angus> moving #include "form_xxx.h" out of the .h files and into the JMarc> Angus> .C files, replacing them with forward declarations of the JMarc> Angus> relevant structs. JMarc> Angus> All files in the src/frontend

Re: forward declarations patch

2000-07-19 Thread Jean-Marc Lasgouttes
> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: Angus> Attached is a small patch to FormPreferences and FormPrint, Angus> moving #include "form_xxx.h" out of the .h files and into the Angus> .C files, replacing them with forward declarations of the Angus> relevant structs. Angus> All

forward declarations patch

2000-07-19 Thread Angus Leeming
Attached is a small patch to FormPreferences and FormPrint, moving #include "form_xxx.h" out of the .h files and into the .C files, replacing them with forward declarations of the relevant structs. All files in the src/frontends/xforms/ directory are now self consistent in this regard. Angus pa