Angus Leeming <[EMAIL PROTECTED]> writes:

|       * most files: "standard-conforming" Licence declaration.
|       Remove trailing whitespace.
>
|       * many header files: do not #include <config.h>
|       Remove unnecessary #includes by using forward declarations.
|       Add as first line: // -*- C++ -*-
|       Move several definitions of virtual functions out of line 
|       as they are unlikely to be inlined anyway (and it reduces
|       header file dependencies)...

Ok, nice.

| Ie, trivial stuff. One further point: many, many of these files 
| (in the qt2 directory) do not have a 
>
| #ifdef __GNUG__
| #pragma interface / implementation
| #endif

| statement. Should I leave things as they are, add a statement to 
| those missing it or remove it from the remaining files, since 
| you've all been quite happily using g++ without any problems.

Leave it for now. It does not seem to make a great difference anymore.

| I'll post the patch to the list once it's finished.

Ok.

-- 
        Lgb

Reply via email to