userdiff: drop parse_driver function
authorJeff King <peff@peff.net>
Wed, 23 Jan 2013 06:25:07 +0000 (01:25 -0500)
committerJunio C Hamano <gitster@pobox.com>
Wed, 23 Jan 2013 16:41:51 +0000 (08:41 -0800)
commit0a5987fe5e0cea23245e0beab6eb47131864b276
tree47181a75625ec08713957c10817916d9b2fb8323
parentd731f0ade129a71237eff5a17f3196002cb439fb
userdiff: drop parse_driver function

When we parse userdiff config, we generally assume that

  diff.name.key

will affect the "key" value of the "name" driver. However,
without confirming that the key is a valid userdiff key, we
may accidentally conflict with the ancient "diff.color.*"
namespace. The current code is careful not to even create a
driver struct if we do not see a key that is known by the
diff-driver code.

However, this carefulness is unnecessary; the default driver
with no keys set behaves exactly the same as having no
driver at all. We can simply set up the driver struct as
soon as we see we have a config key that looks like a
driver. This makes the code a bit more readable.

Signed-off-by: Jeff King <peff@peff.net>
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
userdiff.c