Hmm, there's something odd going on with this pull request. error-face is now calculated value in the the same (let) as error-line-number, error-filename), as least in my branch. I'll push the other changes and hopefully that'll fix it.