DeinAlptraum wrote:

> I don't think str.format() would help here, because it also requires to 
> escape curly braces.

Ah right... good point

> While I've going over the changes once again, I found myself missing a dot in 
> the format string here

I don't see a missing dot there? There's only one between the two placeholders, 
right? In your screenshot it's at the end of the line.

Aside from that, I'm also unsure how I feel about using concatenation just to 
avoid escaping.
I don't have much against escaping, it's only a bit hard to read because it's 
not the backslash one is used to from everywhere else.

https://github.com/llvm/llvm-project/pull/173861
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to