1 |
On 31-08-2011 11:45:27 +0200, Burcin Erocal wrote: |
2 |
> I didn't have time to test the patch myself, since we already have a |
3 |
> report of it not working, here is a justification for my patch... |
4 |
> |
5 |
> AFAICT, the original error message indicates that more arguments are |
6 |
> provided to % than there are place holders in the string. The + |
7 |
> operator has lower precedence than %, so the string is not concatenated |
8 |
> with the part on the previous line. Note that the string in the error |
9 |
> message has 3 place holders only, the fourth should come from the line |
10 |
> above. I added parenthesis around the string addition to fix the |
11 |
> precedence problem. |
12 |
|
13 |
Aha. That explains. Now I understand why the fix fixes it as well. |
14 |
I'll apply it ASAP. Thanks a lot! |
15 |
|
16 |
|
17 |
-- |
18 |
Fabian Groffen |
19 |
Gentoo on a different level |