Hmm, this is a pretty subtle style thing. My way looks better to me, your way looks better to you, I guess :smile: I guess what I worry about with including it on the same line is making all the indents excessively indented. Indeed, re-reading this line I might even suggest pulling self.engine_name all the way back to 4 spaces in below the return, e.g.

```python
    return repr_fmt.format(
        self.engine_name,
        self.keyword_extractor,
        self.link_macro,
        self.charsets)
```

My thought process is that the other way can create some crazy right-indented scenarios that start to conflict with 80 char limit, e.g.

```python
    return repr_fmt.some_long_function().format(self.engine_name,
                                                self.keyword_extractor,
                                                self.link_macro,
                                                self.charsets)
```

see what i mean?