I support breaking lines, actually I was the only one doing it until I saw this. BUT...

Breaking string literals comes at the cost of reducing grep-ability of our code. I personally would write this as:
```
Log(LOG_LEVEL_ERR, 
    "Module protocol was given an overlong +class line (%zd bytes), skipping",
    length);
```
Sometimes string literals can be cleanly broken at specific points, e.g. before the printf modifiers. I rarely do that though, usually I don't break even if the string literal itself does not fit in 78 columns.

Please keep in mind that this is not my own inspiration, I'm used to it after following other big projects' coding style.