bug#40750: Use error face for errors

[ndame]

> I think having Emacs using red for error message in the echo area etc would look pretty odd? Anybody got an opinion here?

I don't think it looks odd, it just makes errors more conspicuous which is not a bad thing:

(run-with-idle-timer

0 nil

(lambda ()

   (message (propertize "Error in post-command-hook (my-test-post-command): (args-out-of-range 0 1)"

                        'face 'error))))

But it can have its own face (error-message) to use by error and signal, so people can customize it.