On Wed, 2006-08-30 at 15:50 +1000, Edward d'Auvergne wrote:
Thanks, these changes look very promising. A few things need to be discussed and tested before merging this code back into the main 1.2 line. I have a few points below interspersed between your code. Firstly I have committed a change which affects the order of initialisation so that the RelaxErrors and RelaxWarnings are initialised after the pedantic flag but before the command line parsing (see https://mail.gna.org/public/relax-commits/2006-08/msg00014.html, Message-id: <E1GIIBN-00026c-Ga@xxxxxxxxxxxxxxxxxx>). This removes the missing AllRelaxErrors error.
Testing this again in preparation for the merge shows that this change has broken the pedantic functionality. The pedantic flag must be correctly set before RelaxWarnings is initialised. Probably the best fix will be to initialise RelaxErrors first, before parsing comand line arguments, then finally initialise the warnings.