RE: [patch] add "underwave" face attribute

[Drew Adams]

> The feature sounds good, I'll let others judge the implementation
> quality, but I have one issue with the "user interface": do we really
> want to let faces specify underline and underwave separately?

No.

> Maybe another approach is to include it as an additional value
> for underline (which can currently be either a boolean or a 
> color name, so we'd have to extend the set of value so as to be
> able to specify shape and color separately).

+1

(And please implement it on other platforms as well.)