Re: [htdig-dev] defaults.cc

[Geoff H. <ghu...@ws...>]

On Sunday, November 10, 2002, at 06:43  PM, Lachlan Andrew wrote:

> While we're changing  Configuration.cc,  what do people
> think about issuing a warning if an attribute is not
> found, rather than silently using the "default_value"
> argument? That would remind developers to add the
> attribute to  defaults.xml  and avoid having default
> values scattered throughout the code...

This would actually be a bit of a problem. While I agree with you that 
this sort of thing would be nice, we've used the Configuration framework 
for parsing text files in new places in 3.2. Notably, the HTML parser 
uses Configuration attributes to parse tags. And of course these won't 
have defaults. :-(

Also, some people declare custom variables in their configuration file, 
which are used for variable substitution, htsearch results, etc.

It might make more sense to:
a) Add this to HtConfiguration, which will avoid problems with HTML.cc
b) Make this a compile-time option, e.g. -DDEBUG_CONFIG

-Geoff