[BUGFIX] Tidy doesn't report invalid options in the config file

• From: J. David Bryan <jdbryan@acm.org>
• Date: Sat, 5 Feb 2000 02:00:05 -0500
• To: HTML Tidy List <html-tidy@w3.org>
• Message-Id: <200002050700.CAA12694@mail.bcpl.net>

This report is for the Tidy version of 13th January 2000.

If an invalid configuration option is specified in the configuration file, 
Tidy silently ignores it.  This is bad because the invalid option may be a 
misspelling of a proper option, e.g., "keep-tine" instead of "keep-time".  
As a result, Tidy may be doing something drastically different than what is 
intended.

The solution is to change line 446 in config.c from:

  NextProperty();

to:

  {
    if (!entry)
      ReportUnknownOption(name);

    NextProperty();
  }

Note that Tidy already reports invalid options that are specified on the 
command line.

                                      -- Dave

Received on Saturday, 5 February 2000 02:00:18 UTC