control characters and Util::clean_text()

[Frédéric Buclin]

> As a result, I think we ought to do one of two things; 1) either treat
> characters that are to be stripped as errors in the user's input, or 2)
> we need to warn users that their input was modified by removing
> unsupported characters.  Acting as if nothing happened is (in my mind)
> unacceptable.


Newlines and other control characters have nothing to do in a bug 
summary, a product version or a target milestone, etc... Silently 
removing these characters sounds fine to me. We are not going to remove 
these characters in comments, nor in product/component/group 
descriptions for instance, and of course not in attachments.

How could you corrupt checkins this way???