Re: [PATCH] cvsserver: Complete rewrite of the configuration parser

On Sat, May 12, 2007 at 12:59:49PM -0700, Junio C Hamano wrote:

Fair enough. So far I see three very minimal solutions, but I can't decide which one is the least ugly:

(For all we can begin by limiting the used variables to ^gitcvs.((ext|pserver).)? )

1) Drop variables named gitcvs.ext and gitcvs.pserver manually
2) Use the complete variable name as key to the hash instead of
   using a hash of hashes of hashes
   { "diff.color => "auto",
     "diff.color.whitespace" => "blue reverse" }
3) Make the second level always a hash, instead of using a string
   directly, so that Junio's example would look like this
   { diff => { color => { value => "auto",
   			  whitespace => "blue reverse" } } }

Opinions?

Gruesse,