Re: diff-function face

[Nick Roberts]

 >     Currently diff-function face inherits from diff-context and looks like
 >     its part of the diff hunk.  I presume this is a mistake as its really
 >     part of the header.  In any case I find a diff is a lot easier to view 
 >     if diff-function inherits from diff-header.
 > 
 > Wouldn't it be better if this were different from the rest of the
 > hunk?  That would help it stand out.  Perhaps it should inherit from
 > some other face, neither diff-header nor diff-function.

More faces doesn't always mean better (I think compilation mode proves that).
It could draw attention away from other equally important parts of the patch.
Either way I don't really mind.  I just think it shouldn't inherit from
diff-context.

-- 
Nick                                           http://www.inet.net.nz/~nickrob